New issue
Advanced search Search tips

Issue 898733 link

Starred by 3 users

Issue metadata

Status: Assigned
Owner:
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Bug



Sign in to add a comment

SignedExchange loading and Service Worker integration

Project Member Reported by horo@chromium.org, Oct 25

Issue description

According to the "Loading Signed Exchanges":
https://wicg.github.io/webpackage/loading.html#overview

```
A Service Worker for https://distributor.example.org/ gets to handle the original request. A Service Worker for https://publisher.example.org/ can then handle the redirect. If it needs to know that signed exchange content is available for the request it’s handling, it has two options:

1. If navigationPreload is enabled, the signed response will be available in the FetchEvent's preloadResponse. Note that this will also cause a network request for requests that aren’t served from a signed exchange.

2. clone() the request and set its cache to "only-if-cached", to retrieve the matching response from either the signed exchange or the HTTP cache. Note that fetch()ing a new Request with the same url will not retrieve the response from the signed exchange.
```

We need to explore how to implement this.
 
Cc: -shimazu@google.com shimazu@chromium.org
Project Member

Comment 2 by bugdroid1@chromium.org, Nov 5

The following revision refers to this bug:
  https://chromium.googlesource.com/chromium/src.git/+/7045466384c91e4078742830c49f6bbe7a5b2d85

commit 7045466384c91e4078742830c49f6bbe7a5b2d85
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Mon Nov 05 02:31:57 2018

Refactor NavigationURLLoaderImpl::URLLoaderRequestController

This CL moves the location where SignedExchangeRequestHandler is created when
NetworkService is disabled from CreateNonNetworkServiceURLLoader() to
StartWithoutNetworkService() where ServiceWorkerRequestHandler is created.

Bug: 898733
Change-Id: If122c3ae296dca7382417f204754ac3ba8f78206
Reviewed-on: https://chromium-review.googlesource.com/c/1298094
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Reviewed-by: Matt Falkenhagen <falken@chromium.org>
Cr-Commit-Position: refs/heads/master@{#605229}
[modify] https://crrev.com/7045466384c91e4078742830c49f6bbe7a5b2d85/content/browser/loader/navigation_url_loader_impl.cc

Sign in to add a comment