Closed
Bug 1734165
Opened 3 years ago
Closed 1 year ago
Add test case for padding in edns response
Categories
(Core :: Networking: DNS, task, P2)
Core
Networking: DNS
Tracking
()
RESOLVED
FIXED
118 Branch
Tracking | Status | |
---|---|---|
firefox118 | --- | fixed |
People
(Reporter: manuel, Assigned: manuel)
References
(Blocks 1 open bug)
Details
(Whiteboard: [necko-triaged])
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
Details |
When adding padding to the request, the response can also contain padding. Add padding in the test case to make sure we handle it correctly
Follow up on https://bugzilla.mozilla.org/show_bug.cgi?id=1543811
See https://phabricator.services.mozilla.com/D127072#inline-698699
Updated•3 years ago
|
Severity: -- → N/A
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [necko-triaged]
Comment 1•3 years ago
|
||
Mmm, I marked this P1 because it's assigned but I guess P2 makes more sense here.
Priority: P1 → P2
Assignee | ||
Comment 2•1 year ago
|
||
Pushed by mbucher@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/13b14aa72b42
Test that padding in edns response doesn't effect parsing edns response r=necko-reviewers,valentin
Comment 4•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox118:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in
before you can comment on or make changes to this bug.
Description
•