richsage / RMSPushNotificationsBundle

NOT MAINTAINED! ⛔️ Push notifications/messages for mobile devices. Supports iOS, Android (C2DM, GCM), Blackberry and Windows Mobile (toast only). A Symfony2 bundle.
MIT License
321 stars 152 forks source link

the AppleNotifications service should handle the case of connection closed by the APNS server #91

Closed stof closed 9 years ago

stof commented 9 years ago

In case the server closes the connection, a subsequent notification will receive an error with code 10 (Shutdown). However, the AppleNotifications service does not handle it and it will continue to reuse the existing stream. This is broken. It should close the corresponding stream and reopen it to initiate a new connection.

richsage commented 9 years ago

This looks to be resolved in #101

stof commented 9 years ago

yeah exactlty (@tgallice works in my team)