If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Make Sentry use a more random identifier for device_app_hash

RESOLVED FIXED

Status

()

Firefox for iOS
General
RESOLVED FIXED
3 months ago
3 months ago

People

(Reporter: st3fan, Assigned: st3fan)

Tracking

unspecified
Other
iOS

Firefox Tracking Flags

(fxios8.1+)

Details

(Whiteboard: [Sentry])

Attachments

(1 attachment)

(Assignee)

Description

3 months ago
The default implementation of device_app_hash is a hash based on the IDFV plus additional properties of the device. We can change this to a (persisistent per install) random identifier that is not connected to iOS provided ID. This will then reset when the app is reinstalled.
(Assignee)

Updated

3 months ago
Whiteboard: [Sentry]
(Assignee)

Updated

3 months ago
Assignee: nobody → sarentz
(Assignee)

Comment 1

3 months ago
Created attachment 8890863 [details] [review]
PR https://github.com/mozilla-mobile/firefox-ios/pull/2983

This patch changes the default `device_app_hash` implementation with our own. The Sentry implementation is based on the *Identifier For Vendors* while our implementation is simply a completely random identifier that is reset on every install.
(Assignee)

Updated

3 months ago
Status: NEW → ASSIGNED
Comment on attachment 8890863 [details] [review]
PR https://github.com/mozilla-mobile/firefox-ios/pull/2983

LGTM. See comment in PR about possibly using `NSUserDefaultsPrefs` instead.
Attachment #8890863 - Flags: review+
(Assignee)

Comment 3

3 months ago
Landed on master, uplifted to v8.x
Status: ASSIGNED → RESOLVED
Last Resolved: 3 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.