Closed Bug 1754744 Opened 3 years ago Closed 3 years ago

Simplifying nsHttpConnection - isolating TLS handshake code

Categories

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

task

Tracking

()

RESOLVED FIXED
101 Branch
Tracking Status
firefox101 --- fixed

People

(Reporter: dragana, Assigned: kershaw)

References

(Blocks 2 open bugs)

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file, 1 obsolete file)

nsHttpConnection drives TLS handshake. Currently, this is mixed in all the other code of nsHttpconnection. It would be good to try to isolate that code into a separate strucure. nsHttpConnection would benefit from some kind of state machine as well so that TLS handshake code and 0RTT code are easier to read.

Assignee: nobody → kershaw
Status: NEW → ASSIGNED
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/36943e34018f isolating TLS handshake code, r=necko-reviewers,dragana
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a7b879827137 isolating TLS handshake code, r=necko-reviewers,dragana

Backed out for causing build bustages on Unified_cpp_protocol_http4.obj

Backout link

Push with failures

Failure log

Failure line: gmake[4]: *** [/builds/worker/checkouts/gecko/config/rules.mk:658: Unified_cpp_protocol_http4.obj] Error 1

Attachment #9271062 - Attachment is obsolete: true
Flags: needinfo?(kershaw)
Pushed by kjang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/926e2d800cbe isolating TLS handshake code, r=necko-reviewers,dragana
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 101 Branch
Depends on: 1764096
Regressions: 1764096
No longer regressions: 1764096
No longer depends on: 1764096
Blocks: 1772202
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: