Closed
Bug 1349246
Opened 8 years ago
Closed 8 years ago
early status events sent nowhere in h2
Categories
(Core :: Networking: HTTP, enhancement)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
mozilla55
| Tracking | Status | |
|---|---|---|
| firefox55 | --- | fixed |
People
(Reporter: u408661, Assigned: u408661)
Details
(Whiteboard: [necko-active][spdy])
Attachments
(1 file)
https://searchfox.org/mozilla-central/source/netwerk/protocol/http/Http2Session.cpp#2310
We're sending early events (dns, connecting) to stream 1, but we never have a real stream 1 (it was skipped because of Upgrade, even though we don't support that for transitioning from h1->h2). We should probably send it to the correct stream.
| Comment hidden (mozreview-request) |
|
||
Comment 2•8 years ago
|
||
| mozreview-review | ||
Comment on attachment 8849589 [details]
Bug 1349246 - fix stream target for early transport status events.
https://reviewboard.mozilla.org/r/122370/#review124510
thanks.. if you could put a comment over where we actually create all the low numbered streams that this is a dependency that would help keep them in sync in the future..
Attachment #8849589 -
Flags: review?(mcmanus) → review+
| Comment hidden (mozreview-request) |
Pushed by hurley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/695436a514db
fix stream target for early transport status events. r=mcmanus
|
||
Comment 6•8 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox55:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in
before you can comment on or make changes to this bug.
Description
•