Closed Bug 1432995 Opened 5 years ago Closed 5 years ago

Sync telemetry should report more detail about http errors


(Firefox :: Sync, enhancement, P1)




Firefox 60
Tracking Status
firefox60 --- fixed


(Reporter: markh, Assigned: tcsc)




(1 file)

Looking at sync telemetry, we see a large number of errors reported as ("othererror", "error.engine.reason.something"). In almost all cases, this tends to mean an unexpected http status code (other than 5xx errors which are reported). This makes it difficult to determine what might be going wrong.

For example, on some days, over 5000 unique devices see "record_upload_fail" doing the history engine, and this doesn't seem to correlate with stats from the server  - having the status code is likely to give us more insights into what the problem actually is. These errors seems transient, but still aren't ideal.

We should even do this for "error.engine.reason.batch_interrupted" - having this reported as ("httperror", 412) gives us the same information.

Note that doing this would mean we lose information about whether the problem was during upload or download. That seems a reasonable tradeoff for a first cut - we could consider adding the http method used in another bug if we found it would be useful.
Priority: -- → P1
Assignee: nobody → tchiovoloni
Comment on attachment 8946381 [details]
Bug 1432995 - Check for HTTP error codes before error strings in the sync ping.

Awesome, thanks!
Attachment #8946381 - Flags: review?(markh) → review+
Pushed by
Check for HTTP error codes before error strings in the sync ping. r=markh
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 60
You need to log in before you can comment on or make changes to this bug.