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

Does stream support iOS 13 push notifications? #89

Closed
EJohnF opened this issue Nov 25, 2019 · 7 comments
Closed

Does stream support iOS 13 push notifications? #89

EJohnF opened this issue Nov 25, 2019 · 7 comments
Assignees

Comments

@EJohnF
Copy link

EJohnF commented Nov 25, 2019

Push notifications were working on friday. Today morning we updated testing phone to iOS 13+ and push notifications stops working:
seems to be related to https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/sending_notification_requests_to_apns?language=objc

In the dashboard I see that push notification claims to be delivered, but I don't see them on the phone

copied from:
GetStream/stream-chat-js#191 (comment)

@EJohnF
Copy link
Author

EJohnF commented Nov 25, 2019

Some my findings which might be helpfull:
I don't see in dashboard push notification logs the entities for Push notifications to ios 13 (no error, not row for this user)
And I made sure that this user is offline - I killed the app where he was logged.

@EJohnF
Copy link
Author

EJohnF commented Nov 26, 2019

from this issue: GetStream/stream-chat-swift#31

I tried to add "apns-push-type":"alert" to the template.

And now I see logs, but it says:

Screenshot 2019-11-26 at 10 47 34

I thought it's because of the wrong key or team ID -> double-checked that it's correct

@bdandy
Copy link

bdandy commented Nov 26, 2019

Hi, @EJohnF, we are having support for new apns push type. It shoudn't be added to the template directly, it's done on backend side.

Are you using development or production push settings?

@EJohnF
Copy link
Author

EJohnF commented Nov 27, 2019

Hi @bogdan-d, thanks for looking up
okey, I've removed this field from the template

I'm uploading .p8 auth key -> which should work for both, development and productions
And the toggle in ios settings -> I tried in both states (on and off for development toggle)

for testing, I'm using AdHoc production build

One more I've noticed, that I'm receiving logs now, about the error during push sending, but the message is different within the same set-up (the app is killed, and certificates/settings are the same):

  • publishing the message failed with error: BadDeviceToken
  • publishing the message failed with error: InvalidProviderToken

@bdandy
Copy link

bdandy commented Nov 27, 2019

Hi @EJohnF

We are currently having the issue with p8 keys you are talking about, working on fix for that.
I recommend you temporary to use .p12 key instead.

Can you check push notifications are working now?

@EJohnF
Copy link
Author

EJohnF commented Nov 27, 2019

Hm, I did nothing special (all settings still the same, and still with p8 key) - and notification start to come on ios13 as expected :/

Seems the issue with p8 is resolved? Or I shouldn't rely on this way?

@bdandy
Copy link

bdandy commented Nov 27, 2019

Issue with p8 is different issue, not related to the topic.

Closing this one

@bdandy bdandy closed this as completed Nov 27, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants