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

Service Worker subresource filter #630

Closed
noahlemen opened this issue May 6, 2021 · 1 comment
Closed

Service Worker subresource filter #630

noahlemen opened this issue May 6, 2021 · 1 comment
Assignees
Labels
Resolution: too early Possibly too early for review or not enough info provided Review type: CG early review An early review of general direction from a Community Group Topic: networking Topic: Service Workers Venue: WICG

Comments

@noahlemen
Copy link

noahlemen commented May 6, 2021

Ya ya yawm TAG!

I'm requesting a TAG review of Service Worker subresource filter.

Service Worker fetch events are currently all-or-nothing – if a Service Worker with a fetch event is registered, all fetches will be intercepted by it. This makes incremental adoption difficult because it subjects all requests to the overhead introduced by the Service Worker even if it doesn't end up handling the request in any meaningful way. A subresource filter will allow a document to specify what subresources are allowed to be intercepted by the Service Worker.

Further details:

You should also know that...

  • We’re interested in conducting an origin trial with the minimal fragment-based version of the API. If the performance data from that trial is positive, then we’d like to move forward with implementing a broader API with more sophisticated URL matching
    • For Facebook, this feature is significantly easier to make use of with fragments, so we’d really like to have the final API still support filtering by fragment.

We'd prefer the TAG provide feedback as (please delete all but the desired option):

💬 leave review feedback as a comment in this issue and @-notify @noahlemen

@plinss
Copy link
Member

plinss commented May 13, 2021

@ylafon and I looked at this in today's breakout session. This seems too early for a TAG review. Please ping us (or file a new review request) once this gets more developed.

Our first-glance feedback is that filtering based on URL fragments does not seem like a viable approach, and we question the use of HTTP headers rather than augmenting the existing scoping mechanism used during Service Worker registration.

@plinss plinss closed this as completed May 13, 2021
@plinss plinss added Resolution: too early Possibly too early for review or not enough info provided and removed Progress: breakout labels May 13, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Resolution: too early Possibly too early for review or not enough info provided Review type: CG early review An early review of general direction from a Community Group Topic: networking Topic: Service Workers Venue: WICG
Projects
None yet
Development

No branches or pull requests

4 participants