1619584 - TRR: Support redirect for TRR request channel

Status: RESOLVED FIXED

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

Description

[Kershaw Chang [:kershaw]]

Current implementation of TRR channel (bug 1613873) is not support redirection. We need to add this feature.

Comment 1

[Kershaw Chang [:kershaw]]

Attachment: Bug 1619584 - P1: Introduce TRRLoadInfo for TRRServiceChannel

Comment 2

[Kershaw Chang [:kershaw]]

Attachment: Bug 1619584 - P2: Implement redirection

Depends on D66772

Comment 3

[Kershaw Chang [:kershaw]]

Attachment: Bug 1619584 - Allow TRR to append DNS query string

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/563470b21a8f
P1: Introduce TRRLoadInfo for TRRServiceChannel r=valentin
https://hg.mozilla.org/integration/autoland/rev/330cffde7320
Allow TRR to append DNS query string r=valentin
https://hg.mozilla.org/integration/autoland/rev/0f77b2c11966
P2: Implement redirection r=valentin,dragana

Comment 5

[Cristian Brindusan [:cbrindusan]]

Backed out 3 changesets (Bug 1619584) for build bustages at UniquePtr.h.
https://hg.mozilla.org/integration/autoland/rev/8f7aea32241c3e439379af55d3e228a26feb7e41

Push with failures:
https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=pending%2Crunning%2Csuccess%2Ctestfailed%2Cbusted%2Cexception&revision=0f77b2c1196611c96e3289de3455dd031c510aeb&selectedJob=293525114

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

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/f9566be72ee5
P1: Introduce NullLoadInfo for TRRServiceChannel r=valentin
https://hg.mozilla.org/integration/autoland/rev/d9cc82da0150
Allow TRR to append DNS query string r=valentin
https://hg.mozilla.org/integration/autoland/rev/264d787ca2e7
P2: Implement redirection r=valentin,dragana

Comment 7

[Andrei Ciure[:aciure]]

Backed out 3 changesets (bug 1619584) for causing build bustages
https://hg.mozilla.org/integration/autoland/rev/9ec828023a8266996b9969de733f6a48b3b0cae2

push that caused the backout: https://treeherder.mozilla.org/#/jobs?repo=autoland&resultStatus=testfailed%2Cbusted%2Cexception%2Cretry%2Cusercancel%2Crunnable&revision=264d787ca2e76db2e0ef2c92497892ddc3779a5c&failure_classification_id=2

Comment 8

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/b2610da59077
P1: Introduce TRRLoadInfo for TRRServiceChannel r=valentin
https://hg.mozilla.org/integration/autoland/rev/52cfa33cd009
Allow TRR to append DNS query string r=valentin
https://hg.mozilla.org/integration/autoland/rev/400e2c6564c5
P2: Implement redirection r=valentin,dragana

Comment 9

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/b2610da59077
https://hg.mozilla.org/mozilla-central/rev/52cfa33cd009
https://hg.mozilla.org/mozilla-central/rev/400e2c6564c5