search

appfeel / node-pushnotifications

Push notifications for GCM, APNS, MPNS, AMZ (automatic detection from device token)
MIT License
534 stars 126 forks source link

[Bug] GCM tokens recognition problem #57

Closed olegmdev closed 6 years ago

olegmdev commented 6 years ago

GCM push tokens may include both amzn and adm substrings (just got one). So in fact you will send GCM token to ADM and it'll be failed.

https://github.com/appfeel/node-pushnotifications/blob/master/src/push-notifications.js#L89

alex-friedl commented 6 years ago

Duplicate of #53 ?

olegmdev commented 6 years ago

@alex-friedl right, closing this one