1673554 - Add an option to override the Ion id in `submitExternalPing`

Status: RESOLVED FIXED

(Firefox :: Pioneer, task, P1)

Description

[Alessio Placitelli [:Dexter]]

From this issue:

With the new architecture the Ion id is stored in the Core addon. Unfortunately this means that TelemetryController.submitExternalPing will throw unless it is also set in the toolkit.telemetry.pioneerId pref.

To keep both about:ion and Ion Core working when it comes to telemetry, we will need to add an overridePioneerId option (like overrideClientId) that, if present, will use the provided id and no check the pref at all.

Comment 1

[Alessio Placitelli [:Dexter]]

Attachment: Bug 1673554 - Add an option to override the pioneer id. r?chutten

With this change, calling the submitExternalPing API
will not necessarily require the pioneer id value to be
stored in a pref. This added flexibility will allow the
Pioneer/Ion Core Addon to own the storage of the id,
communicating it to the legacy telemetry APIs as needed.

Comment 2

[MarniePW [:marnie]]

[Tracking Requested - why for this release]: We'd like to get this into Firefox for the 84 release.

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/9651640bcc07
Add an option to override the pioneer id. r=chutten

Comment 4

[Alexandru Michis [:malexandru]]

https://hg.mozilla.org/mozilla-central/rev/9651640bcc07