Open
Bug 1755518
Opened 3 years ago
Updated 6 months ago
nsHttpConnection state machine
Categories
(Core :: Networking: Proxy, enhancement, P3)
Core
Networking: Proxy
Tracking
()
NEW
People
(Reporter: dragana, Unassigned)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
- 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)
Comment 4•8 months ago
|
||
Moving bug to Core/Networking: Proxy.
Component: Networking: HTTP → Networking: Proxy
Updated•6 months ago
|
Flags: needinfo?(valentin.gosu)
You need to log in
before you can comment on or make changes to this bug.
Description
•