Closed
Bug 1250902
Opened 9 years ago
Closed 9 years ago
Push notifications from http://simple-push-demo.appspot.com/ stopped working
Categories
(Core :: DOM: Notifications, defect, P1)
Tracking
()
RESOLVED
WORKSFORME
Tracking | Status | |
---|---|---|
firefox47 | --- | affected |
People
(Reporter: janx, Unassigned)
References
Details
(Whiteboard: btpp-followup-2016-03-03)
Attachments
(1 obsolete file)
STR:
1. Go to
2. Activate "Enable Push Notifications" (and authorize permission prompt)
3. Click on "Send a Push [...]" button
Expected:
- A push notification with the weather appears.
Actual result:
- No notification is ever shown.
From the Browser Console:
PushClient.setUpPushPermission() main.js:16:4837
subscriptionUpdate: PushSubscription { endpoint: "https://updates.push.services.mozil…" } main.js:16:842
"curl --request POST https://updates.push.services.mozilla.com/push/gAAAAABWyvwk7v4mcVwNx6NkwQ_nGqzbDMf9uC81QsH7fsu4rvQ1UG93PumefhsBNJBrsEG71D-3gEWJEFpdZiYPdxD908U-NOl2bRLP7Nslye4Sb2XaroUs-DEub5W6_KpeelYeiDZ-uRMg_S9xPbluSnqxKgIH7eNlzbx96P47ZsjKS5qG5Yk=" main.js:16:1310
endpoint: "https://updates.push.services.mozilla.com/push/gAAAAABWyvwk7v4mcVwNx6NkwQ_nGqzbDMf9uC81QsH7fsu4rvQ1UG93PumefhsBNJBrsEG71D-3gEWJEFpdZiYPdxD908U-NOl2bRLP7Nslye4Sb2XaroUs-DEub5W6_KpeelYeiDZ-uRMg_S9xPbluSnqxKgIH7eNlzbx96P47ZsjKS5qG5Yk=" main.js:16:1612
Fetch Error :-S Error: Unsuccessful attempt to send push message
Stack trace:
r/i/</<@https://simple-push-demo.appspot.com/scripts/main.js:16:1757
promise callback*r/i/<@https://simple-push-demo.appspot.com/scripts/main.js:16:1640
EventListener.handleEvent*r/i@https://simple-push-demo.appspot.com/scripts/main.js:16:1438
[2]</s</<.value/<@https://simple-push-demo.appspot.com/scripts/main.js:16:5198
promise callback*[2]</s</<.value@https://simple-push-demo.appspot.com/scripts/main.js:16:4885
promise callback*e@https://simple-push-demo.appspot.com/scripts/main.js:16:4340
EventListener.handleEvent*[1]<@https://simple-push-demo.appspot.com/scripts/main.js:16:2789
r@https://simple-push-demo.appspot.com/scripts/main.js:16:246
e@https://simple-push-demo.appspot.com/scripts/main.js:16:421
@https://simple-push-demo.appspot.com/scripts/main.js:16:11
Reporter | ||
Updated•9 years ago
|
Comment 1•9 years ago
|
||
The demo on that page doesn't use the Web Push protocol but the proprietary GCM, so I think this never worked in Firefox.
Comment 2•9 years ago
|
||
No, it does support Web Push as well: https://github.com/gauntface/simple-push-demo-backend/blob/master/main.py
Maybe the issue is that it isn't sending the TTL header?
Reporter | ||
Comment 3•9 years ago
|
||
I used to trigger push events on http://simple-push-demo.appspot.com/, so it used to work before.
When looking for other demos, https://goroost.com/try-web-push also doesn't seem to work anymore.
Maybe a regression?
Comment 4•9 years ago
|
||
(In reply to Jan Keromnes [:janx] from comment #3)
> I used to trigger push events on http://simple-push-demo.appspot.com/, so it
> used to work before.
The TTL requirement was added very recently, so this would explain why it was working before and isn't working now. I've submitted a pull request to add the TTL header to requests: https://github.com/gauntface/simple-push-demo-backend/pull/1.
https://goroost.com/try-web-push doesn't seem to load for me, so I can't test it.
Comment 5•9 years ago
|
||
Kit, this is just the TTL change, right?
Flags: needinfo?(kcambridge)
Whiteboard: btpp-followup-2016-03-03
Comment 6•9 years ago
|
||
I think so. Marco has a link to the PR in comment 4. Also, today's deploy should make things easier by relaxing the TTL requirement if the browser is connected.
Flags: needinfo?(kcambridge)
Reporter | ||
Comment 7•9 years ago
|
||
Marking as fixed because the TTL requirement was relaxed and both demos work again.
Thanks Marco for submitting that PR! This should be a more permanent fix.
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Reporter | ||
Comment 8•9 years ago
|
||
Re-opening because the demo is broken again in Nightly.
STR:
1. Go to https://simple-push-demo.appspot.com/
Expected:
- You can activate "Enable Push Notifications" and trigger a Push Notification.
Actual:
- The "Enable Push Notifications" option is greyed, and the page shows "Ooops a Problem Occured".
Error details from the page:
[Exception... "Method not implemented" nsresult: "0x80004001 (NS_ERROR_NOT_IMPLEMENTED)" location: "JS frame :: https://simple-push-demo.appspot.com/scripts/main.js :: [2]</s</<.value :: line 16" data: no]
From the BrowserConsole:
PushClient.setUpPushPermission() main.js:16
PushClient.setUpPushPermission() Error <unavailable> main.js:16
<unavailable> main.js:16
Doesn't reproduce in Chrome.
However, other demos like https://goroost.com/try-web-push still work.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Reporter | ||
Updated•9 years ago
|
Attachment #8723018 -
Attachment is obsolete: true
Comment 9•9 years ago
|
||
This is caused by bug 1261405, but I would open a new bug to track the regression.
Comment 10•9 years ago
|
||
The demo was removed, but the issue was definitely fixed.
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → WORKSFORME
You need to log in
before you can comment on or make changes to this bug.
Description
•