New issue
Advanced search Search tips

Issue 850008 link

Starred by 1 user
Status: Fixed
Owner:
horo@chromium.org
Closed: Jun 2018
Cc:
kinuko@chromium.org
kouhei@chromium.org
ksakamoto@chromium.org
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 3
Type: Bug



Sign in to add a comment

'Disable cache' in DevTools doesn't affect on the certificate requests of Signed Exchange.

Project Member Reported by horo@chromium.org, Jun 6 2018 
Chrome Version: TOT 69.0.3450.0

What steps will reproduce the problem?
(1) Enable chrome://flags/#enable-signed-http-exchange
(2) Open https://htxg-b1.appspot.com/sxg/hello_ec.sxg
(3) Open DevTools and check 'Disable cache' flag.
(4) Open https://htxg-b1.appspot.com/sxg/hello_ec.sxg again.

What is the expected result?
The certificate request must bypass the cache.

What happens instead?
The certificate "ec256" for the second load is served from the cache.

Comment 1 by horo@chromium.org, Jun 6 2018

Description: Show this description
Project Member

Comment 2 by bugdroid1@chromium.org, Jun 7 2018 

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

commit 6e6782db92e68ea6fd6384f67d3d54490bad714d
Author: Tsuyoshi Horo <horo@chromium.org>
Date: Thu Jun 07 02:32:47 2018

Set force_fetch flag of cert fetcher when the original request load flags has LOAD_BYPASS_CACHE

Bug:  850008 
Change-Id: I9da3323ced6f8dae7ee8725f248762b0321fbd68
Reviewed-on: https://chromium-review.googlesource.com/1088452
Reviewed-by: Kunihiko Sakamoto <ksakamoto@chromium.org>
Reviewed-by: Kinuko Yasuda <kinuko@chromium.org>
Commit-Queue: Tsuyoshi Horo <horo@chromium.org>
Cr-Commit-Position: refs/heads/master@{#565151}
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/loader/navigation_url_loader_impl.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/loader/prefetch_url_loader.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/loader/prefetch_url_loader.h
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/signed_exchange_handler.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/signed_exchange_handler.h
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/signed_exchange_handler_unittest.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/web_package_loader.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/web_package_loader.h
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/web_package_prefetch_handler.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/web_package_prefetch_handler.h
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/web_package_request_handler.cc
[modify] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/content/browser/web_package/web_package_request_handler.h
[add] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/third_party/WebKit/LayoutTests/http/tests/devtools/htxg/htxg-disable-cache-expected.txt
[add] https://crrev.com/6e6782db92e68ea6fd6384f67d3d54490bad714d/third_party/WebKit/LayoutTests/http/tests/devtools/htxg/htxg-disable-cache.js

Comment 3 by horo@chromium.org, Jun 7 2018

Status: Fixed (was: Started)
►

Sign in to add a comment