Closed Bug 1931321 Opened 1 year ago Closed 1 year ago

Figure out where the 'version' annotation is coming from and remove it

Categories

(Toolkit :: Crash Reporting, defect)

defect

Tracking

()

RESOLVED INVALID

People

(Reporter: gsvelto, Unassigned)

References

Details

I've just noticed that recent crash reports include an annotation called 'version' which is not the 'Version' annotation (that is the product version) and appears to always be set to the value '2'. Here's a couple of examples:

https://crash-stats.mozilla.org/report/index/0ae96ba3-0cf1-49dc-96b0-98ea00241114#tab-annotations
https://crash-stats.mozilla.org/report/index/9e6a1ed7-9c1c-4b6f-9191-ea0830241114#tab-annotations

This seems to be happening both on desktop Firefox and in Fenix, so it's probably an issue somewhere in the crash generation machinery.

Will, I've been digging around a bit and it seems that this annotation comes from antenna (in particular it seems to me it was introduced in bug 1687987). Is it expected for it to show up in the Annotations tab?

Flags: needinfo?(willkg)

The collector adds several fields at collection:

https://antenna.readthedocs.io/en/latest/overview.html#collector-added-fields

You're right in that it's weird these show up in the annotations tab. We can fix that so it's clearer which are annotations from the crash reporter and which are fields added by the collector.

Want me to do that work in this bug or write up another?

Flags: needinfo?(willkg) → needinfo?(gsvelto)

Let's make a separate bug in the appropriate component. I opened it here because I thought it was a regression on our part since we've been replacing tons of client-side crash reporting code. And BTW it's not urgent now that I know where it's coming from.

Status: NEW → RESOLVED
Closed: 1 year ago
Flags: needinfo?(gsvelto)
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.