Closed
Bug 960727
Opened 10 years ago
Closed 10 years ago
[meta] Refresh Webmaker auth system
Categories
(Webmaker Graveyard :: Login, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: brett, Assigned: cade)
References
Details
Attachments
(4 files)
Based on the architecture of the new events and profile apps, we should reconsider our auth solution.
Comment 2•10 years ago
|
||
Initial thoughts on a new login system, done with cade and kate: https://etherpad.mozilla.org/newwebmakerlogin
Comment 3•10 years ago
|
||
As soon as it makes sense, that etherpad's contents should get copied into this bug so it doesn't get lost/disconnected from this bug. Also, what you're proposing looks interesting.
Comment 4•10 years ago
|
||
I did some rough work on an implementation on the client-side part of our ideas here (no tests or AMD): https://github.com/k88hudson/webmaker-auth-client. I wanted to test: - Saving tokens/user data in local storage - Restoring state instantly from a previous session and silently verifying - Managing errors and login/logout state with events I'd also like to try adding the create a new user panel etc. which I suspect will be the most challenging aspect of the front-end, but I figured I wouldn't go too far before we do back-end work.
Comment 5•10 years ago
|
||
Lots and lots of background research on Oauth 1.0 and Oauth 2.0 done by Kate: https://docs.google.com/document/d/1m5kZsHwPuDNPwSrdBX4kb7pUHUd4RrhYAKB8AUQvQSc/edit# Notes on what the new login system will have in terms of API responses: https://docs.google.com/document/d/165CpDWlNPXEfU5LK4KcjdvgZGHvlJr2zhau8NAXBQCg/edit#
Status: NEW → ASSIGNED
Summary: Refresh Webmaker auth system → [meta] Refresh Webmaker auth system
Whiteboard: preworkweek
Comment 6•10 years ago
|
||
Comment 7•10 years ago
|
||
Comment 8•10 years ago
|
||
Adding my PR until I file the bug for it
Comment 9•10 years ago
|
||
Comment 11•10 years ago
|
||
Oh look, all the dependencies are cleared.
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•