Closed Bug 1473333 Opened 3 years ago Closed 3 years ago

Log success rates with telemetry probe in HttpChannel onstartrequest


(Core :: Networking: HTTP, enhancement, P1)




Tracking Status
firefox63 --- fixed


(Reporter: bagder, Assigned: bagder)


(Whiteboard: [necko-triaged][trr])


(3 files)

With a probe there that logs the HTTP success rate at this point, we should be able to compare error rates for users using TRR with those that don't use TRR.
To measure success rate in nsHttpChannel::OnStartRequest

MozReview-Commit-ID: Jb2ixouu6zU

New data collections are subject to Data Collection Review by a Data Steward:

(It's just a short "form" to fill out, then getting an r? from a Steward)
Comment on attachment 8989918 [details]
bug 1473333 - add Telemetry probe HTTP_CHANNEL_ONSTART_SUCCESS

Patrick McManus [:mcmanus] has approved the revision.
Attachment #8989918 - Flags: review+
Data Collection Review for Telemetry::HTTP_CHANNEL_ONSTART_SUCCESS
Attachment #8990215 - Flags: review?(francois)
Comment on attachment 8990215 [details]

1) Is there or will there be **documentation** that describes the schema for the ultimate data set available publicly, complete and accurate?

Yes, in Histograms.json.

2) Is there a control mechanism that allows the user to turn the data collection on and off?

Yes, telemetry setting.

3) If the request is for permanent data collection, is there someone who will monitor the data over time?**

Yes, Daniel Stenberg.

4) Using the **[category system of data types](** on the Mozilla wiki, what collection type of data do the requested measurements fall under?  **

Category 1.

5) Is the data collection request for default-on or default-off?

Default ON, only in pre-release channels.

6) Does the instrumentation include the addition of **any *new* identifiers** (whether anonymous or otherwise; e.g., username, random IDs, etc.  See the appendix for more details)?


7) Is the data collection covered by the existing Firefox privacy notice?


8) Does there need to be a check-in in the future to determine whether to renew the data? (Yes/No)

No, permanent.
Attachment #8990215 - Flags: review?(francois) → review+
(Unable to land this in lando, bug 1473810, I'm taking this route instead)
Attachment #8989918 - Attachment is obsolete: true
Attachment #8990656 - Flags: review+
Keywords: checkin-needed
Pushed by
add Telemetry probe HTTP_CHANNEL_ONSTART_SUCCESS r=mcmanus
Keywords: checkin-needed
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
Attachment #8989918 - Attachment is obsolete: false
You need to log in before you can comment on or make changes to this bug.