search

jazzband / django-push-notifications

Send push notifications to mobile devices through GCM or APNS in Django.
MIT License
2.28k stars 618 forks source link

Fix #744: Bulk messages with aoiapns not working properly #746

Open mikaelengstrom opened 2 days ago

mikaelengstrom commented 2 days ago

Previous version implemented aioapns in a way that made it hang indefinetly. Especially when receiver list contaned a lot of bad tokens.

Having a lot of bad tokens still affects the reliability and transfer speed of notification sets, which is why this fix also deactivate devices for error codes BadDeviceToken and DeviceTokenNotForTopic unlike previous versions.

Tagging @pomali who initially did the aoiapns integration in case you have any feedback.