Open
Bug 1755518
Opened 3 years ago
Updated 2 years ago
nsHttpConnection state machine
Categories
(Core :: Networking: HTTP, enhancement, P3)
Core
Networking: HTTP
Tracking
()
NEW
People
(Reporter: dragana, Unassigned, NeedInfo)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
|
(deleted),
text/x-phabricator-request
|
Details |
- Bug 1754737 did the first step and added states to nsHttpConnection. it is:
- UNINITIALIZED,
- SETTING_UP_TUNNEL,
- TUNNEL_DONE,
- TUNNEL_NOT_USED
This can be extended to : - UNINITIALIZED,
- TLS_FOR_TUNNEL,
- SETTING_UP_TUNNEL,
- TLS_TO_ORIGIN,
- EARLY_DATA,
- FINAL_REQUEST
We do not need separate TUNNEL_DONE and TUNNEL_NOT_USED, we can use the HttpConnectionInfo to find out whether a proxy was used.
- There s some code that is H2-only and some that are HTTP<2-only. We should try to split this into 2 sub-structures, helper classes, to make the code more readable.
|
Reporter | |
Updated•2 years ago
|
|
|
Reporter | |
Updated•2 years ago
|
Type: task → enhancement
|
|
Reporter | |
Updated•2 years ago
|
Summary: Add more structure to nsHttpConnection → nsHttpConnection state machine
|
|
Reporter | |
Updated•2 years ago
|
Assignee: nobody → dd.mozilla
Status: NEW → ASSIGNED
|
|
Reporter | |
Comment 1•2 years ago
|
||
|
|
Reporter | |
Comment 2•2 years ago
|
||
I will not have time to work on this bug, so I am unassigning myself.
Assignee: dd.mozilla → nobody
Status: ASSIGNED → NEW
|
|
Reporter | |
Comment 3•2 years ago
|
||
Valentin, please take a look at the patch if you want to finish it or just abandon it. Thank you.
Flags: needinfo?(valentin.gosu)
You need to log in
before you can comment on or make changes to this bug.
Description
•