Register device with FxA before obtaining a signed certificate

NEW
Unassigned

Status

()

Firefox for Android
Firefox Accounts
P3
normal
a year ago
3 months ago

People

(Reporter: Grisha, Unassigned)

Tracking

Firefox Tracking Flags

(Not tracked)

Details

(Reporter)

Description

a year ago
Side-effect of calling /certificate/sign is creation of a placeholder fxa device record if one isn't already present. From rfkelly ([0]):

> * If we get a request to /certificate/sign, and
> * That sessionToken does not have a device id associated with it, and
> * There's no ?service= query parameter, or there's service=sync, then
> * Create a device record for that sessionToken, because it's pretty definitely a 
> device using sync

We should attempt to perform FxA device registration before calling /certificate/sign, if possible. That call happens as part of the "Cohabiting" state action, right before we move into "Married" state ("good to sync!"). Device registration currently happens once we're in the "Married" state. See [1] and [2].

[0] https://bugzilla.mozilla.org/show_bug.cgi?id=1332858#c7
[1] https://dxr.mozilla.org/mozilla-central/source/mobile/android/services/src/main/java/org/mozilla/gecko/fxa/login/Cohabiting.java?q=path%3ACohabiting.java&redirect_type=single#27
[2] https://dxr.mozilla.org/mozilla-central/source/mobile/android/services/src/main/java/org/mozilla/gecko/fxa/sync/FxAccountSyncAdapter.java#541
(Reporter)

Updated

a year ago
See Also: → bug 1332858
Bulk edit: moving potentially relevant issues from disabled Android Background Services product to Firefox for Android.

Removing priority to throw back to triage owners: please close these bugs if they're no longer relevant! Thanks!
Component: Firefox Accounts → Firefox Accounts
Product: Android Background Services → Firefox for Android

Updated

3 months ago
Priority: -- → P3
You need to log in before you can comment on or make changes to this bug.