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

Update AndroidGCMNotification.php #57

Closed DarwinOnLine closed 10 years ago

DarwinOnLine commented 10 years ago

There is a last problem with MultiCurl and Windows, form Android GCM notifications. To complete the issue https://github.com/richsage/RMSPushNotificationsBundle/issues/31, and the PR https://github.com/richsage/RMSPushNotificationsBundle/pull/54, I added the deactivation of peer verification, like in https://github.com/richsage/RMSPushNotificationsBundle/blob/master/Service/OS/AndroidNotification.php#L75

Hope that helps :)

DarwinOnLine commented 10 years ago

Exact @richsage, my bad :(

richsage commented 10 years ago

Thanks - I think it would be good if this was configurable at some point, but if it makes it work for Windows users then let's get it in for now :-)