Skip to content
This repository has been archived by the owner on Feb 15, 2020. It is now read-only.

Add special case for FxOS FTU pings (bug 1210894) #175

Merged
merged 1 commit into from Dec 16, 2015

Conversation

mreid-moz
Copy link
Contributor

Most FTU pings are being sent to the error stream due to missing the
expected 'info' section of the payload. These pings can be identified
by 'ver = 3' (this is either 1 or 2 for normal old-style Telemetry
records). Example FTU v3 doc:

{
  "ver": 3,
  "screenHeight": 1080,
  "screenWidth": 1920,
  "devicePixelRatio": 1,
  "locale": "en-GB",
  "pingID": "deadbeef-0000-0000-95b0-f387d780548f",
  "activationTime": 1449740000000,
  "deviceinfo.os": "2.1.0.0",
  "deviceinfo.software": "Boot2Gecko 2.1.0.0",
  "deviceinfo.platform_build_id": "20151210052433",
  "deviceinfo.platform_version": "34.0",
  "deviceinfo.product_model": "PTV-TX-58DX780E",
  "deviceinfo.firmware_revision": "0001",
  "deviceinfo.hardware": null,
  "app.update.channel": "default",
  "pingTime": 1449740000000
}

Most FTU pings are being sent to the error stream due to missing the
expected 'info' section of the payload. These pings can be identified
by 'ver = 3' (this is either 1 or 2 for normal old-style Telemetry
records). Example FTU v3 doc:

{
  "ver": 3,
  "screenHeight": 1080,
  "screenWidth": 1920,
  "devicePixelRatio": 1,
  "locale": "en-GB",
  "pingID": "deadbeef-0000-0000-95b0-f387d780548f",
  "activationTime": 1449740000000,
  "deviceinfo.os": "2.1.0.0",
  "deviceinfo.software": "Boot2Gecko 2.1.0.0",
  "deviceinfo.platform_build_id": "20151210052433",
  "deviceinfo.platform_version": "34.0",
  "deviceinfo.product_model": "PTV-TX-58DX780E",
  "deviceinfo.firmware_revision": "0001",
  "deviceinfo.hardware": null,
  "app.update.channel": "default",
  "pingTime": 1449740000000
}
@mreid-moz
Copy link
Contributor Author

It's a little easier to see what actually changed if you stick ?w=1 on the end of the diff URL

rafrombrc added a commit that referenced this pull request Dec 16, 2015
Add special case for FxOS FTU pings (bug 1210894)
@rafrombrc rafrombrc merged commit 0e462fd into mozilla-services:master Dec 16, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants