Closed Bug 1343008 Opened 3 years ago Closed 3 years ago

Fix -> h2, tls1.3 early-data and POST conbination

Categories

(Core :: Networking: HTTP, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: dragana, Assigned: dragana)

Details

(Whiteboard: [necko-active])

Attachments

(1 file)

No description provided.
Assignee: nobody → dd.mozilla
Status: NEW → ASSIGNED
Whiteboard: [necko-active]
bool
Http2Stream::Do0RTT()
{
  MOZ_ASSERT(mTransaction);
  mAttempting0RTT = true;
  return mTransaction->Do0RTT();
}

should not set mAttempting0RTT state to true if trans->Do0RTT() fails (as it will currently for non-safe methods)
Attachment #8841696 - Flags: review?(mcmanus)
Attachment #8841696 - Flags: review?(mcmanus) → review+
Keywords: checkin-needed
https://hg.mozilla.org/integration/mozilla-inbound/rev/5dc23fac58010d61efe9733da71177afcaa2e0e8
Bug 1343008 - Fix combination of H2, tls1.3 early-data and not safe http methods. r=mcmanus
https://hg.mozilla.org/mozilla-central/rev/5dc23fac5801
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
You need to log in before you can comment on or make changes to this bug.