Closed fabianuribe closed 6 years ago
It could also be my demo falling out of date or doing something funky, so please feel free to raise issues on simple-push-demo if so.
Thanks @gauntface, we are also observing the same behavior on our end, I've decided to reference your awesome demo as quick way to reproduce the issue. I will make sure to open an issue on simple-push-demo
if it turns out that we are both out of date.
Same "push" event handler works for Chrome and Samsung Internet Browser Beta 7.4, whereas Samsung Internet Browser 6.4 is going into fallback mode for Yahoo.com notifications.
Thanks for raising this @fabianuribe, just to let you know that I've shared this with our colleagues and we're going to check on it.
Hi again @fabianuribe. Hopefully some good news for you on this: our new stable release v7.2 is just being rolled out to everyone now over the next few days (we'll be announcing in a blog post early next week). I just checked this issue in v7.2 and thankfully it appears fixed (same as v7.4):
Closing as fixed now, but if you spot any further problems then of course please let us know :)
It appears there might be a regression introduced on the Push Notifications support for current version of Samsung Internet Browser.
At least on
6.4.10.5
the Push Notification payload is missing thedata
field, whereas on the Beta version7.4.00.46
is still working correctly. A quick way to reproduce is to use @gauntface 's simple push demo (https://gauntface.github.io/simple-push-demo/) and test it by sending a push with some custom message:When attempting to parse the the
data
field of the Push Notification event, an exception is thrown (Cannot read property 'json' of null
) which suggests there might be an issue decrypting the payload coming from GCM/FCM.cc/ @poshaughnessy