Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.Edited this issue because my FF mobile was way out of date.
1. Subscribe to notifications on Chrome mobile, Firefox mobile. Verify they appear in /admin/config/pwa/notification/subscriptions
2. Send notification
3. On /admin/config/pwa/notification/subscriptions only 1 of them (chrome) is marked with "last used"
(chrome mobile, firefox desktop, chrome desktop - all work!)
Not sure if this is an issue with pwa_webpush or the library.
| Comment | File | Size | Author |
|---|---|---|---|
| #6 | Screenshot_20201008_113802.png | 72.14 KB | djdevin |
Comments
Comment #2
djdevinComment #3
djdevinComment #4
djdevinComment #5
nod_So you see expired subscription in the
admin/config/pwa/notification/subscriptions/expiredpage?Comment #6
djdevinSo it's not expiring, I had that issue initially but I believe my firefox mobile was way out of date.
The issue is it's just not sending, I'm not sure why. Nothing in the watchdog except "Notification sent to users: 1, 598, 599. " Sorry I haven't had time to dig into it yet, here's a SS (after I sent notifications to all users). The "last used" for the firefox mobile isn't filled in.
admin = regular firefox - received
firefoxdevin = firefox on mobile (android) - not received
chromedevin = chrome on mobile (android) - received
minishlink/web-push v6.0.2
Comment #7
djdevinHuh:
{"code": 500, "errno": 999, "error": "Internal Server Error", "more_info": "http://autopush.readthedocs.io/en/latest/http.html#error-codes", "message": "Server error: INVALID_ARGUMENT: Request contains an invalid argument."}Comment #8
djdevinAfter investigation I found https://github.com/web-push-libs/web-push-php/issues/108
Setting
fixed it for FF Android and other browsers worked as well.
Comment #9
nod_like written here: https://github.com/web-push-libs/web-push-php/issues/108#issuecomment-54... let's add a default padding of 2000, i'm worried having this padding size would signal the use of the PWA module but is that better or worst than not padding message? I don't know. Might need to add a configuration for that actually.
Comment #11
nod_Thanks for looking into it. Reworked the admin page a bit and added a setting for the padding size. It now defaults to 2847 so it should be safe for firefox mobile :)