mirror of
https://github.com/binwiederhier/ntfy.git
synced 2026-05-09 08:26:00 +02:00
[GH-ISSUE #461] iOS App not receiving any notifations with 'Priority' set #352
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#352
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 @pinpox on GitHub (Nov 1, 2022).
Original GitHub issue: https://github.com/binwiederhier/ntfy/issues/461
Not sure if it's a duplicate of #353, but the iOS app fails to receive any notications that have
Priorityset.Example:
This is not received on my mobile phone
While this works as expected:
I've tried different Priority values, they all seem to break the delivery on iOS. Subscribing to the topic via the CLI shows everything fine, it seems to be related to iOS.
@binwiederhier commented on GitHub (Nov 3, 2022):
log-level: TRACEset? Just to rule out that this is a server-side issue?@0x111 commented on GitHub (Nov 13, 2022):
I can confirm that this is happening in my case too. The desktop version displays both, while the ios app only the one without the priority flag.
ntfy version: ntfy 1.28.0 (d3cfa34), runtime go1.18.6, built at 2022-09-27T16:55:57ZI did remove ip addresses and instance urls, but this is the Trace logs. I was sending a ntfy message from the same server where ntfy is running momentarily. I also used the same messages as provided by the OP.
If you need more cooperation or logs @binwiederhier let me know, I am happy to help, because now any message with priority flag, does not pop up on my ios app which was apparently working for me until a few days ago.
Edit:
2. No cloudflare for me, although I use reverse proxy which I have set the flag as defined in docs
3. Done here
4. Done
@mpeterson commented on GitHub (Sep 9, 2023):
I started to try ntfy today and notice issue is still prevalent almost a year after first reporting. I'm using it through cloudflare and can confirm that notifications with priority are never seen by the iOS client. I think this will make iOS user's adoption quite hard sadly, as it's harder to trust a system that doesn't work the way it should.
@binwiederhier commented on GitHub (Sep 9, 2023):
@mpeterson ntfy is my side project. I work on the things that are fun for me, and of course I try to fix issues as they arise. I deployed a fix for the priority header issue as soon as I learned it was an issue as best I could. Apparently it doesn't work perfectly, but you can always use the
X-Priorityheader instead and it should work just fine.If you are not happy with a feature, you can always fork ntfy and fix it yourself. Complaining about it is certainly not helpful.
@mpeterson commented on GitHub (Sep 9, 2023):
@binwiederhier sorry if I came across as rude. I didn't meant to complain, but clearly I chose a poor writing. I meant to raise awareness to the fact that this is still not working (as there haven't been many updates on the ticket) and that might make onboarding and getting traction of the project harder for iOS users.
What you've done is loable and I truly appreciate it, again sorry for my poor choice of wording.
@binwiederhier commented on GitHub (Sep 9, 2023):
No worries. I am not an iOS user. I made the iOS app as best as I can, and there are many known issues. I have tried many many times to recruit iOS devs, without success.
I don't think this is an iOS issue though. I'm not sure though.
@Spyros-Amvrakidis commented on GitHub (Oct 12, 2023):
Only for integration with Uptime Kuma and NTFY (Both behind Cloudflare)
I have rewrite Uptime Kuma Config to send only notifications without Priority
Here is the Config and the Custom Deployment for ntfy.js in Uptime Kuma
ConfigMap
`
apiVersion: v1
kind: ConfigMap
metadata:
name: ntfy-config
data:
ntfy.js: |
const NotificationProvider = require("./notification-provider");
const axios = require("axios");
const { DOWN, UP } = require("../../src/util");
`
Deployment just add this 2 Volumes
This is under
spec:
volumes:
`
configMap:
name: ntfy-config
`
and this is under
containers:
volumeMounts:
`
mountPath: /app/server/notification-providers
`
I know its not the best solution but it works
This config based on K3S and K8S infra but probably can work with some changes in docker compose etc
@binwiederhier if you can edit it to look normal please do that, as I can't put all configMap in same code block and I have write the volumes overwrite outside of code block as I try multiple times and it didn't join correctly in code block
@tcpsyn commented on GitHub (Nov 6, 2023):
Another way to work around this is to use the Apprise notification handler in Uptime Kuma. ntfy:://server/topic or ntfys:://server/topic for ssl.
@binwiederhier commented on GitHub (Nov 20, 2023):
The Priority header fix has been deployed for v2.8.0 (server) now. I re-read the ticket and I cannot make sense of it. We do not forward the Priority header to APNS when talking to the upstream server (ntfy.sh), and the header is in no way shape or form involved when fetching the notification from the selfhosted server.
I also manually tried it myself with a local server (not behind cloudflare obviously), and that also worked like a charm.
I am baffled how this should only affect iOS and nothing else.
If you use the
X-Priorityheader instead ofPriority, does that work?