Closed Bug 1429859 Opened 3 years ago Closed 3 years ago

Due to a bug we sometime send a request that got 425 as early-data again

Categories

(Core :: Networking: HTTP, defect, P1)

58 Branch
defect

Tracking

()

RESOLVED FIXED
mozilla59
Tracking Status
firefox58 --- fixed
firefox59 --- fixed

People

(Reporter: dragana, Assigned: dragana)

References

Details

(Whiteboard: [necko-triaged])

Attachments

(1 file)

No description provided.
Assignee: nobody → dd.mozilla
Status: NEW → ASSIGNED
Attachment #8941909 - Flags: review?(mcmanus)
Attachment #8941909 - Flags: review?(mcmanus) → review+
Keywords: checkin-needed
Priority: -- → P1
Whiteboard: [necko-triaged]
Comment on attachment 8941909 [details] [diff] [review]
bug_1429859.patch

Approval Request Comment
[Feature/Bug causing the regression]: bug 1406908
[User impact if declined]: If firefox send some request with tls1.3 early-data, it may happen that server replies with 425 Too Early. Firefox should resend the request but not as early-data. Due to this bug firefox sometimes resend the request in early-data. In that case user will see error 425 as a response.
[Is this code covered by automated tests?]: no
[Has the fix been verified in Nightly?]: no
[Needs manual test from QE? If yes, steps to reproduce]: no 
[List of other uplifts needed for the feature/fix]:none
[Is the change risky?]: no, it is a one line patch
[Why is the change risky/not risky?]:
[String changes made/needed]:
Attachment #8941909 - Flags: approval-mozilla-beta?
Blocks: 1406908
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/mozilla-inbound/rev/9d7620e3e096
Check mDoNotTryEarlyData in Do0RTT(). r=mcmanus
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/9d7620e3e096
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Comment on attachment 8941909 [details] [diff] [review]
bug_1429859.patch

Fix an issue which will cause server to reply with 425 too early. Beta58+
Attachment #8941909 - Flags: approval-mozilla-release+
Attachment #8941909 - Flags: approval-mozilla-beta?
Attachment #8941909 - Flags: approval-mozilla-beta+
Duplicate of this bug: 1429221
You need to log in before you can comment on or make changes to this bug.