New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 731303 link

Starred by 1 user

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Linux , Android , Windows , Chrome , Mac
Pri: 1
Type: Bug



Sign in to add a comment

retry_without_alt_svc_on_quic_errors does not work with pooled sessions

Project Member Reported by rch@chromium.org, Jun 8 2017

Issue description

If requests for domain1 are pooled onto a connection to domain2, and those requests fail without the connection closing, then QUIC will be marked as broken to domain2. However, the connection domain2 will remain active. Requests with an origin of domain2 will correctly not be served via QUIC. However requests for domain1 will still be sent to this connection where they will fail.
 

Comment 2 by rch@chromium.org, Jun 15 2017

Labels: Merge-Request-60
Project Member

Comment 3 by sheriffbot@chromium.org, Jun 15 2017

Labels: -Merge-Request-60 Hotlist-Merge-Review Merge-Review-60
This bug requires manual review: M60 has already been promoted to the beta branch, so this requires manual review
Please contact the milestone owner if you have questions.
Owners: amineer@(Android), cmasso@(iOS), josafat@(ChromeOS), bustamante@(Desktop)

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Please apply appropriate OSs.  Thanks.

Comment 5 by rch@chromium.org, Jun 19 2017

Labels: OS-Android OS-Chrome OS-Linux OS-Mac OS-Windows
Labels: -Merge-Review-60 Merge-Approved-60
This CL will allow QUIC to failover to TCP to prevent future urgent respins.  It's relatively small and well tested from what I can tell on the CL, and no further merges anticipated.  Approved for M60 branch 3112.
Project Member

Comment 7 by sheriffbot@chromium.org, Jun 23 2017

Cc: amineer@chromium.org
This issue has been approved for a merge. Please merge the fix to any appropriate branches as soon as possible!

If all merges have been completed, please remove any remaining Merge-Approved labels from this issue.

Thanks for your time! To disable nags, add the Disable-Nags label.

For more details visit https://www.chromium.org/issue-tracking/autotriage - Your friendly Sheriffbot
Labels: -Merge-Approved-60 merge-merged-3112
This already landed here: https://chromium.googlesource.com/chromium/src.git/+/d4087f89dfc467197e431974d2e7e603b7b9d43e
rch@, should we mark as fixed?

Comment 10 by rch@chromium.org, Jun 23 2017

Status: Fixed (was: Started)
Indeed. Good point.

Sign in to add a comment