1641222 - Follow CNAME/AliasForm chains

Status: RESOLVED FIXED

(Core :: Networking: DNS, defect, P2)

Comment 1

[Valentin Gosu [:valentin] (he/him)]

Attachment: Bug 1641222 - Follow CNAME/AliasForm chains r=dragana

Comment 2

[Dragana Damjanovic [:dragana]]

A general question that can be a follow up: I think we should structure trr a bit more. As an example what about trr request returns a response and does not follow CNAME or AliasForm.. And we have another layer that decides if it return a result (in case of A/AAAA ) answer or if it creates a new trr query. Basically to split TRR.cpp into 2 classes a simple request/response and a "recursive resolver"

Comment 3

[Valentin Gosu [:valentin] (he/him)]

I'll try to do that in bug 1635174.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4e775c2385b2
Follow CNAME/AliasForm chains r=dragana,necko-reviewers

Comment 5

[Pulsebot]

Backout by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a2bb355e129f
Backed out changeset 4e775c2385b2 for causing failure at test_trr_httpssvc_wrap.js. CLOSED TREE

Comment 6

[Atila Butkovits]

Backed out for causing failure at test_trr_httpssvc_wrap.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/a2bb355e129f224817f9f3e694c43594e00988d3

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&selectedTaskRun=cxEHwa7_T_WelgaeVIxjmA.0&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunning%2Cpending%2Crunnable&revision=4e775c2385b2df56b00eddb2f32474818abc54e6

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=306473890&repo=autoland&lineNumber=3558

Comment 7

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/ce1a126dcf10
Follow CNAME/AliasForm chains r=dragana,necko-reviewers

Comment 8

[Atila Butkovits]

Backed out for failures at test_trr_httpssvc.js.

Backout link: https://hg.mozilla.org/integration/autoland/rev/12a7de90bb95a7d654d8d2e013a6effe5bb983e2

Push with failures: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception&revision=ce1a126dcf105e348ab891cf285a7ca73b295dcb&selectedTaskRun=bTHM6DseQsyIO91SEA74Sw.0

Failure log: https://treeherder.mozilla.org/logviewer.html#/jobs?job_id=307218599&repo=autoland&lineNumber=3368

Comment 9

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/083212102e96
Follow CNAME/AliasForm chains r=dragana,necko-reviewers

Comment 10

[Atila Butkovits]

https://hg.mozilla.org/mozilla-central/rev/083212102e96