Open
Bug 1772190
Opened 2 years ago
Updated 8 months ago
nsHttpConnection should be responsible for reading CONNECT response
Categories
(Core :: Networking: Proxy, enhancement, P3)
Core
Networking: Proxy
Tracking
()
NEW
People
(Reporter: dragana, Unassigned)
References
(Depends on 1 open bug, Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
nsHttpConnection is responsible for creating the CONNECT request and sending it, but it is not responsible for reading. This is done in 2 different ways:
- For HTTP/1.1 proxy: nsHttpTransaction is reading and parsing the response. Then nsHttpConnection::OnHeadersAvailable is called to handle the response.
- For HTTP/2 proxy: Http2Stream(Http2Stream::ConvertResponseHeaders) is responsible. If the response is > 299, e.g. 407, it iss propagated to the transaction to handle it.
Recommendation:
nsHttpConnection should read the response to the CONNECT request and inform the nsHttpTransaction.
Comment 1•8 months ago
|
||
Moving bug to Core/Networking: Proxy.
Component: Networking: HTTP → Networking: Proxy
You need to log in
before you can comment on or make changes to this bug.
Description
•