Closed Bug 1352772 Opened 5 years ago Closed 5 years ago
Permissions not correctly transmitted to processes which load Service Workers
During the testing of Bug 1352764, Bug 1352765 and Bug 1352769, I detected a problem in Nightly 55 (2017-04-01). It didn't happen in Nightly (2017-03-20). One specific push notification demo now doesn't work. Here's how to reproduce the bug: 1. Open https://gauntface.github.io/simple-push-demo/ 2. Enable Push Notifications on the page 3. Add some payload text and click "Send a push via XHR" button Result: Notification doesn't appear (tested on clear profile) Expected: Notification should appear. Sometimes it appears, very rarely
Yea, stopped working for me as well when I upgraded from March 30 to April 1 nightly FF55. I'm on travel at the moment and can't look at it for a few days. Kit, any ideas?
The reason why this is occurring is because the ServiceWorker is being loaded in a process which has not loaded gauntface.github.io yet. This means that the permissions for that domain haven't been sent down yet. This should have been caught by my assertions, but I messed up when implementing the assertion and the assertions don't fire in this specific case. I've filed bug 1353179 to fix this assertion. To fix this specific problem, I need to send down permissions to the content process loading the ServiceWorker whenever the ServiceWorker is loaded. I don't currently understand the events which occur during the creation of a ServiceWorker. If there is a convenient place where we are in the parent process, and know the principal of the ServiceWorker which we are just about to ask to load a ServiceWorker, that would be a great place to send the permissions down. bkelly, is there a convenient place like that?
Assignee: nobody → michael
Component: DOM: Push Notifications → DOM: Service Workers
Flags: needinfo?(michael) → needinfo?(bkelly)
Summary: Push notifications in one specific demo stopped working (Nightly 2017-04-01) → Permissions not correctly transmitted to processes which load Service Workers
This patch should send down the permissions to the content process before starting the Service Worker in it. Like we talked about on IRC, this is mostly a stopgap solution which will need to be changed in the future when we get a single ServiceWorker per origin per firefox instance. MozReview-Commit-ID: DVLjumi9iBJ
Attachment #8854529 - Flags: review?(catalin.badea392)
Comment on attachment 8854529 [details] [diff] [review] Send permissions to the content process before dispatching push events Review of attachment 8854529 [details] [diff] [review]: ----------------------------------------------------------------- lgtm.
Attachment #8854529 - Flags: review?(catalin.badea392) → review+
Sorry I was on travel during this discussion. Looks like its been taken care of, though. Thanks!
Pushed by email@example.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/0f19bf1a2ce6 Send permissions to the content process before dispatching push events, r=catalinb
You need to log in before you can comment on or make changes to this bug.