1615443 - Re-add Telemetry for DTLS protocol version

Status: RESOLVED FIXED

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

Description

[Nils Ohlmeier [:drno]]

In bug 1607283 we removed the DTLS protocol telemetry probe, because it had expired. But with the landing off DTLS 1.3 in bug 1284103 it would be helpful to know how much usage 1.3 gets compared to 1.2.

Comment 1

[Nils Ohlmeier [:drno]]

Attachment: Bug 1615443: re-add DTLS protocol version telemetry. r=dminor

Comment 2

[Nils Ohlmeier [:drno]]

Try run: https://treeherder.mozilla.org/#/jobs?repo=try&revision=88dc495321a8627b5eb8899bb1858f7fd31cf83

Comment 3

[BugBot [:suhaib / :marco/ :calixte]]

There's a r+ patch which didn't land and no activity in this bug for 2 weeks.
:drno, could you have a look please?
For more information, please visit auto_nag documentation.

Comment 4

[Nils Ohlmeier [:drno]]

Attachment: data-request.txt

Comment 5

[Megan McCorquodale]

Comment on attachment 9134194 [details]
data-request.txt

1. Is the provided Data Collection Review complete, correct, and data-review+ by a Data Steward?
Yes.

2.  Is the data collection covered by the existing Firefox Privacy Notice?
Yes.

-----
data-review +

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ead3b8dc11c
re-add DTLS protocol version telemetry. r=dminor

Comment 7

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/6ead3b8dc11c

Comment 8

[Byron Campen [:bwc]]

Did you verify whether this works when the socket process is enabled?

Comment 9

[Ryan VanderMeulen [:RyanVM]]

Do we need to uplift this to Beta for Fx75?

Comment 10

[Nils Ohlmeier [:drno]]

(In reply to Ryan VanderMeulen [:RyanVM] from comment #9)

Do we need to uplift this to Beta for Fx75?

I don't think we need to. We will learn in 76 how many sites are still using DTLS 1.0. As we don't have immediate plans to remove it soon (depends on how the pandemic plays out) I think we can simply wait.

Comment 11

[Nils Ohlmeier [:drno]]

(In reply to Byron Campen [:bwc] from comment #8)

Did you verify whether this works when the socket process is enabled?

I had not verified it.
But I do see it showing up in Nightly here about:telemetry#histograms-tab_socket
I interpret it that it's working.