Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Silent/Background notifications don't work for P8 and Apns2 #598

Open
shved270189 opened this issue Mar 15, 2021 · 5 comments
Open

Silent/Background notifications don't work for P8 and Apns2 #598

shved270189 opened this issue Mar 15, 2021 · 5 comments
Labels

Comments

@shved270189
Copy link
Contributor

Silent/Background notification works fine for legacy Apns provider. After update to P8 or Apns2 Silent/Background notifications don't work. I checked apple documentation: https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/pushing_background_updates_to_your_app#2980040 about background notifications APNs server requires the apns-push-type field when sending push notifications to Apple Watch, and recommends it for all platforms. Also from documentation about POST APNS requests: https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/sending_notification_requests_to_apns#2947607 apns-push-type header required for watchOS 6 and later; recommended for macOS, iOS, tvOS, and iPadOS.

This header is recommended for iOS, but without it Background notification does not work fine.

I added this header in this PR #596 and it helps me to fix this issue.

have anyone other ideas, solutions for this issue?

@benlangfeld
Copy link
Collaborator

Fix merged in #596

@benlangfeld
Copy link
Collaborator

Once we merge a solution to #568 we can cut v6.0.0 with this fix in it.

@aried3r
Copy link
Member

aried3r commented Mar 18, 2021

I want to reference this comment #596 (comment) here since we might need to change a few things in order for this to ship in 6.0. But I'd need some help from someone who uses APNs.

@aried3r
Copy link
Member

aried3r commented Apr 22, 2021

Hey @shved270189, @rpush/maintainers. I'd love to release v6 of rpush soon and for this I'd love to see the questions I had in #596 addressed if possible. I assume APNs would still work without these fixes, but I'd be nice if someone who actively uses APNs and maybe has a bit more insight could have a look.

Thanks!

@aried3r
Copy link
Member

aried3r commented May 21, 2021

I have released a version 6.0.0 which contains the fix from #596 but I'm leaving this open for now.

@aried3r aried3r added the APNs label May 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants