Open Bug 1772190 Opened 2 years ago Updated 2 years ago

nsHttpConnection should be responsible for reading CONNECT response

Categories

(Core :: Networking: HTTP, enhancement, P3)

Product:
Core
Component:
Networking: HTTP
Type:
enhancement

Tracking

()

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.

Depends on: 1772191
You need to log in before you can comment on or make changes to this bug.