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)

task

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)

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

Assignee: nobody → mbucher
Depends on: 1543811
Severity: -- → N/A
Status: NEW → ASSIGNED
Priority: -- → P1
Whiteboard: [necko-triaged]

Mmm, I marked this P1 because it's assigned but I guess P2 makes more sense here.

Priority: P1 → P2
Blocks: 1734579
No longer blocks: 1734579
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
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 118 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: