Closed Bug 892050 Opened 11 years ago Closed 11 years ago

Add a Persona account manager to Settings

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect)

x86_64
Gonk (Firefox OS)
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED INVALID

People

(Reporter: jedp, Assigned: jedp)

References

Details

Attachments

(2 files, 1 obsolete file)

With FTE sign-in with Persona coming up for v1.2, I think we should add Persona account management to Settings.

Such an account manager should have feature parity with the persona.org dashboard (sign in to https://login.persona.org/ to view).

It think it should also provide a way to sign out everywhere on device.
Implements "logout everywhere"
Assignee: nobody → jparsons
Implements logout everywhere.  Stubs out other account management elements.
added missing file
Attachment #775043 - Attachment is obsolete: true
Comment on attachment 775131 [details] [diff] [review]
gaia part: a pref to set the persona uri

Review of attachment 775131 [details] [diff] [review]:
-----------------------------------------------------------------

::: apps/settings/js/persona.js
@@ +14,5 @@
> +      return;
> +    }
> +
> +    settings.createLock().set({
> +      'identity.logout-everywhere': Date.now()

The use of settings to put a message in a mailslot for chrome to receive (by observing 'mozsettings-changed') is not appropriate as a final solution because the message channel is not confidential, nor is it secure.  Chrome has no way of knowing whether these messages are really coming from the Settings app.
Bug 876397 proposes an Inter-App Communication API.  We could use this to message to the system app, and from the system app, send mozContentEvents down to chrome.
Depends on: inter-app-comm-api
We are no longer planning Persona sign-in on FTU; instead, we will be using Firefox Accounts.

There's a Firefox Accounts manager coming in the Settings app (bug 929388).  I think that's what we want here.

Closing as invalid.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → INVALID
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: