mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[GH-ISSUE #382] Android: UnifiedPush subscription not using saved user credentials #294
Labels
No labels
ai-generated
android-app
android-app
android-app
🪲 bug
build
build
dependencies
docs
enhancement
enhancement
🔥 HOT
in-progress 🏃
ios
prio:low
prio:low
pull-request
question
🔒 security
server
server
unified-push
web-app
website
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/ntfy#294
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @ryester19 on GitHub (Aug 17, 2022).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/382
Using Ntfy server 1.27.2, Android Ntfy app 1.13.0, and Android Element 1.4.32
ACL is enabled on the server, and auth-default-access set to 'deny-all'
The subscription that Element sets up for UnifiedPush use does not use the saved user credentials in the ntfy app. The user is an admin, and the "Managed users" page does say the user is being used for this topic. The only error I see in the server log is a "unauthorized: unauthorized" message
In order to get Element to pass the "Test Push" test, I have to give anonymous read-write access to the topic that's created by Element
Android log:
https://nopaste.net/PFREHjyS0Q
@MaximilianGaedig commented on GitHub (Aug 18, 2022):
Just ran into the same issue, the workaround for now is using the ntfy.sh server for unified push and your own for other things but that's no real workaround as you are not using your server
@binwiederhier commented on GitHub (Aug 18, 2022):
This is not a bug in ntfy, but rather a design limitation of UnifiedPush (UP). In UP, the URL is the secret, user/password cannot be used in the spec.
This explains this in more detail: https://unifiedpush.org/users/distributors/ntfy/#limit-access-to-some-users
Specifically this section:
@binwiederhier commented on GitHub (Aug 18, 2022):
For more questions about UP, please check out the UP Matrix room(s)
@ryester19 commented on GitHub (Aug 18, 2022):
Gotcha. To get around this limitation in a more user-friendly way, could we add a ACL sub-option in the config file or an alias toggle in the 'ntfy access' command?
@binwiederhier commented on GitHub (Aug 29, 2022):
@ryester19 Any suggestions?
@binwiederhier commented on GitHub (Sep 3, 2022):
Let me know if you have any suggestions. For now I'll close this issue.