Alt-Svc: clear does not work with canonical hosts |
|||||||||
Issue descriptionIf a server emits Alt-Svc: clear, chrome correctly removes the Alt-Svc mapping for this host. However, if there is a canonical host for this server, that entry will still apply, thus defeating the clear. This makes it impossible to disable QUIC via Alt-Svc.
,
Jun 20 2016
,
Jun 20 2016
Your change meets the bar and is auto-approved for M52 (branch: 2743)
,
Jun 22 2016
@rch: Are there any manual steps to verify the above fix ?
,
Jun 22 2016
Weird, I merged this to m52 in https://codereview.chromium.org/2080803002/, but the bug was not updated. Did I do something wrong? [m52] Make Alt-Svc: clear work with canonical hosts BUG= 620482 Review-Url: https://codereview.chromium.org/2069203003 Cr-Commit-Position: refs/heads/master@{#400060} (cherry picked from commit 0c49a120616a28296dd457a023b63a65be94a15b) Committed: https://chromium.googlesource.com/chromium/src/+/788b5d73328a22fc8233a48e00d5a804aa711153 Patch Set 1 (edit)
,
Jun 22 2016
ashejole: manual verification is a bit tricky because it requires a server to be emitting the Alt-Svc: clear header which is not currently happening. We noticed this when some of our servers needed to disable QUIC. But QUIC was re-enabled so we're not emitting this header any longer.
,
Jun 22 2016
I see the CL merged in to M52 branch on the branch tip. https://chromium.googlesource.com/chromium/src.git/+log/refs/branch-heads/2743?n=1000 @Tina, can you please check why the bug is not updated with the merge information ?
,
Jun 22 2016
The comment updating is done via bugdroig script (not merge script), + sheyang@ to take a further look.
,
Jun 22 2016
Unfortunately bugdroid encountered an exception when processing this commit in all attempts: SSLError: [Errno 1] _ssl.c:1429: error:1408F119:SSL routines:SSL3_GET_RECORD:decryption failed or bad record mac The build is https://uberchromegw.corp.google.com/i/internal.infra.cron/builders/bugdroid/builds/5441/steps/bugdroid/logs/stdio
,
Jun 22 2016
thanks for quick looking! Given not a way to backfill the commit via bugdroid, I've fixed the merge label. rch@ pls feel free to copy your merge CL into this bug as well.
,
Jun 29 2016
,
Aug 16 2016
|
|||||||||
►
Sign in to add a comment |
|||||||||
Comment 1 by bugdroid1@chromium.org
, Jun 16 2016