Closed
Bug 1215724
Opened 9 years ago
Closed 9 years ago
Brotli doesn't work over SPDY
Categories
(Core :: Networking: HTTP, defect)
Tracking
()
RESOLVED
FIXED
mozilla44
Tracking | Status | |
---|---|---|
firefox44 | --- | fixed |
People
(Reporter: piotrsikora, Assigned: mcmanus)
Details
(Whiteboard: [spdy])
Attachments
(1 file)
1.30 KB,
patch
|
u408661
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_10_5) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/45.0.2454.101 Safari/537.36 Steps to reproduce: Connect to a server (localhost, sorry) running nginx-1.9.4 (with SPDY/3.1) with ngx_brotli module. Actual results: Firefox didn't send "accept-encoding" header (even though it's displayed in Developer Tools). It was fine to skip it when only gzip was supported, since SPDY spec requires clients to support it, but now that Brotli is also supported, Firefox needs to send it. Expected results: Firefox should send "accept-encoding: gzip, deflate, br" header.
Assignee | ||
Comment 1•9 years ago
|
||
I guess so - for the most part spdy is waiting to die and we don't add new features to it.. but in this case the feature shouldn't be transport version specific. thanks for noticing.
Assignee: nobody → mcmanus
Whiteboard: [spdy]
Assignee | ||
Comment 2•9 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=fe1f6a46a5d9
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8675295 -
Flags: review?(hurley)
Updated•9 years ago
|
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #8675295 -
Flags: review?(hurley) → review+
Assignee | ||
Comment 4•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/c23714f8078b2fe7665438daa6217867490568b1 bug 1215724 - enable brotli on spdy r=hurley
Comment 5•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/c23714f8078b
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
status-firefox44:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla44
You need to log in
before you can comment on or make changes to this bug.
Description
•