Closed
Bug 1741422
Opened 4 years ago
Closed 4 years ago
Write tests for 103 EarlyHints receive over HTTP/2
Categories
(Core :: Networking: HTTP, task, P3)
Core
Networking: HTTP
Tracking
()
RESOLVED
FIXED
96 Branch
| Tracking | Status | |
|---|---|---|
| firefox96 | --- | fixed |
People
(Reporter: dragana, Assigned: dragana)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 external file)
No description provided.
|
Assignee | |
Comment 1•4 years ago
|
||
Update: our HTTP/2 implementation propagates 1xx response already, therefore there is no need to change code to make this work.
I tried to make tests but nodejs does not support sending 103 responses.
I will see if there is another server that we can use.
Assignee: dd.mozilla → nobody
Status: ASSIGNED → NEW
Priority: P2 → P3
Summary: Make 103 EarlyHints observer work with HTTP/2 → Write tests for 103 EarlyHints receive over HTTP/2
|
|
Assignee | |
Comment 2•4 years ago
|
||
Reducing priority to p3, because the most parts of the code will be tests using the http/1.1 and http/3 server. There is only a small piece of the code that is untested.
|
|
Assignee | |
Comment 3•4 years ago
|
||
GetHeader can return NS_ERROR_NOT_AVAILABLE which is not a fatal error and we should not assert.
|
||
Updated•4 years ago
|
Assignee: nobody → dd.mozilla
Status: NEW → ASSIGNED
Pushed by ddamjanovic@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/aa2a07bf1eb1
Add tests for the 103 responses over HTTP/2 r=necko-reviewers,valentin
|
||
Comment 5•4 years ago
|
||
| bugherder | ||
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox96:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 96 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•