Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Drop executionContexts? #59

Open
annevk opened this issue May 3, 2019 · 4 comments

Comments

Projects
None yet
4 participants
@annevk
Copy link
Member

commented May 3, 2019

We might drop this feature from Firefox as Chrome doesn't implement it and we keep running into issues with it. Latest is https://bugzilla.mozilla.org/show_bug.cgi?id=1548034.

@wanderview

This comment has been minimized.

Copy link
Member

commented May 3, 2019

It seems to me we either need to:

  1. Figure out how to make executionContexts work correctly.
  2. Or spec what actually happens to things like open blobs, cache_storage Response, etc when their underlying storage is nuked. AFAIK that is completely undefined.

Reloading contexts seems like the best option to actually getting the user to a known sane state.

@annevk

This comment has been minimized.

Copy link
Member Author

commented May 3, 2019

2 is needed either way right? That seems like a separate problem that also exists.

@asakusuma

This comment has been minimized.

Copy link

commented May 3, 2019

There may be an expectation that executionContexts is required to ensure storage kills a service worker. I would much rather provide the guarantee that storage alone kills any service workers though. Tangentially, there were already some potential issues with spec guarantees around removing service workers: #54

@ericlaw1979

This comment has been minimized.

Copy link

commented May 21, 2019

It appears that Firefox landed their removal: https://bugzilla.mozilla.org/show_bug.cgi?id=1548034#c8

kesselb added a commit to nextcloud/server that referenced this issue Jul 9, 2019

kesselb added a commit to nextcloud/server that referenced this issue Jul 9, 2019

kesselb added a commit to nextcloud/server that referenced this issue Jul 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.