search

activitree / meteor-push

Meteor Push Notifications for Cordova and Web/PWA with Firebase (FCM).
MIT License
27 stars 19 forks source link

iOS 13 bad device token #7

Closed jamauro closed 4 years ago

jamauro commented 4 years ago

Looks like Apple changed something and now I'm seeing Bad Device Token in iOS 13 and this device: {length = 32, bytes = 0x965b251c 6cb1926d e3cb366f dfb16ddd ... 5f857679 376eab7c }

I found this article which has more details: https://nshipster.com/apns-device-tokens/

I'm not sure if this should be resolved in this package or node-apn. Thoughts?

paulincai commented 4 years ago

Hi @jamauro. I think this is related to https://github.com/phonegap/phonegap-plugin-push/issues/2819 . Within the Meteor package we only take what is being provided and save it to Mongo. In /lib/client/cordova.js line 70, PushNotification is coming from the Push Cordova plugin (https://github.com/phonegap/phonegap-plugin-push/blob/master/docs/API.md)

jamauro commented 4 years ago

Ok thanks for pointing me in the right direction!

wkeithchau commented 4 years ago

I think this is related to phonegap/phonegap-plugin-push#2819

It looks like a fix for it has been merged and published with version 2.3.0?

jamauro commented 4 years ago

Confirmed that phonegap-plugin-push@2.3.0 fixes this issue. @paulincai not sure how you'd like to represent that in the Readme.

paulincai commented 4 years ago

Migrated entirely to FCM in V2.