Closed Bug 1349246 Opened 5 years ago Closed 5 years ago

early status events sent nowhere in h2

Categories

(Core :: Networking: HTTP, enhancement)

enhancement
Not set
normal

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 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+
Done, thanks!
Pushed by hurley@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/695436a514db
fix stream target for early transport status events. r=mcmanus
https://hg.mozilla.org/mozilla-central/rev/695436a514db
Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla55
You need to log in before you can comment on or make changes to this bug.