Closed
Bug 922377
Opened 11 years ago
Closed 11 years ago
Authenticate nucleus with django-browserid
Categories
(Websites :: Nucleus, defect, P4)
Websites
Nucleus
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: hoosteeno, Assigned: osmose)
References
Details
(Whiteboard: [kb=1144732] )
Once we have a steel thread of the end-to-end functionality required to publish release notes, let's swap out the authentication mechanism.
Reporter | ||
Updated•11 years ago
|
Whiteboard: [kb=1144732]
Reporter | ||
Updated•11 years ago
|
Assignee: nobody → mkelly
Assignee | ||
Comment 1•11 years ago
|
||
This only applies to logging into the admin interface, correct? We're not thinking of using BrowserID for any sort've API access?
Flags: needinfo?(jmize)
Comment 2•11 years ago
|
||
For read-only access to the API, no authentication is currently needed, as all information is public. For write access to the API, We currently only have token based authentication enabled, but it would be nice if someone could get an API token via a web interface after authenticating via persona. We could also easily enable session-based API access as well, in which case someone could potentially log in via browserid to obtain write access to the API, which could be useful if someone were using the django-rest-framework html views to post information, either as an alternative to the standard django admin or (more likely) as a way of learning about how to write to the API. This could also be useful if we decided to implement a fancy html5 interface that made API calls via ajax down the road.
Flags: needinfo?(jmize)
Assignee | ||
Comment 3•11 years ago
|
||
That sounds like stuff we'd add in the future, so I propose keeping the scope of this particular bug to adding django auth to the admin interface. Agreed?
Comment 4•11 years ago
|
||
Commit pushed to master at https://github.com/mozilla/nucleus https://github.com/mozilla/nucleus/commit/7cee4d7997fea4364e1f0ff8a4bb1b7c4f3cbd1a Bug 922377: Support logging into the admin interface with Persona. Currently points towards a django-browserid branch with un-merged code. Once the feature lands in django-browserid's master branch we'll update the submodule to reference it.
Reporter | ||
Comment 5•11 years ago
|
||
This bug is resolved and merged into the nucleus/rna codebase.
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•