1460314 - TRR: add telemetry probe for used HTTP version

Status: RESOLVED FIXED

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

Comment 1

[Patrick McManus [:mcmanus]]

nudge

Comment 2

[Mahak]

@bagder I would like to work on this .
could you please elaborate more :)

Comment 3

[(inactive) Nihanth Subramanya [:nhnt11]]

Attachment: Bug 1460314 - TRR: add telemetry probe for used HTTP version.

Comment 4

[(inactive) Nihanth Subramanya [:nhnt11]]

Made a patch due to holiday boredom. Not sure we actually want to land this. Our DoH strategy is focused around TRR and shipping good defaults, I don't understand the benefit of having a pref to force http2-only or whatever. (I'm assuming that's the only reason for collecting this telemetry)

Comment 5

[Dragana Damjanovic [:dragana]]

the idea was to disable TRR if HTTP2 or higher is not used. I do no think we will do that now. If HTTP < 2 is used the performance will be really bad, but I am not sure we want to disable TRR. The code complexity that that would involved is not worth it.
We can look at the data anyway.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/084e0dcdaae2
TRR: add telemetry probe for used HTTP version. r=necko-reviewers,valentin

Comment 7

[Sandor Molnar[:smolnar]]

https://hg.mozilla.org/mozilla-central/rev/084e0dcdaae2