h2 tunnel streams queued for reaching session parallelism limits are not dequeued
Categories
(Core :: Networking: HTTP, defect, P1)
Tracking
()
People
(Reporter: mayhemer, Assigned: mayhemer)
References
Details
(Whiteboard: [necko-triaged][secure-proxy][skyline])
Attachments
(1 file)
|
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
P3 because when we have bug 1562885, this is WONTFIX. Still, should be at least tracked.
|
Assignee | |
Comment 1•5 years ago
|
||
Looks like the reason we are broken here is that we consume the CONNECT request stream before we check for concurrency room. The request data then gets lost and when the stream is dequeued (that actually works) we don't have anything to write.
|
|
Assignee | |
Comment 2•5 years ago
|
||
|
||
Updated•5 years ago
|
|
|
Assignee | |
Updated•5 years ago
|
|
|
||
Updated•5 years ago
|
|
|
Assignee | |
Comment 5•5 years ago
|
||
This bug is real. On some heavier websites linking h1 end points, or reddit and proxy per-ses parallelism limited to 3 streams, I'm getting indefinite hangs. W/ the fix (and checking that it hits the code) there are no hangs at all.
|
||
Updated•5 years ago
|
Pushed by honzab.moz@firemni.cz: https://hg.mozilla.org/integration/autoland/rev/3cc56185d9bc Properly dequeue tunnel streams that were queued because of high concurrency, r=dragana
|
||
Comment 7•5 years ago
|
||
| bugherder | ||
|
|
Assignee | |
Comment 8•5 years ago
|
||
Comment on attachment 9075650 [details]
Bug 1563027 - Properly dequeue tunnel streams that were queued because of high concurrency, r=dragana
Beta/Release Uplift Approval Request
- User impact if declined: Network requests hanging, sometimes until browser restart, when h2 proxy is configured, see Comment 5 for details.
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: Yes
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): It's hard to assess for me, but this code is hit quite often, so we would already find problems with just Nightly coverage.
- String changes made/needed:
|
||
Comment 9•5 years ago
|
||
Comment on attachment 9075650 [details]
Bug 1563027 - Properly dequeue tunnel streams that were queued because of high concurrency, r=dragana
Fixes hung network streams in some circumstances. Approved for 69.0b14.
|
|
||
Comment 10•5 years ago
|
||
| bugherder uplift | ||
Description
•