Closed
Bug 1411373
Opened 8 years ago
Closed 8 years ago
Auth0 Authentication Support
Categories
(Conduit :: Lando, enhancement)
Conduit
Lando
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: smacleod, Assigned: smacleod)
Details
Attachments
(4 files)
|
45 bytes,
text/x-phabricator-request
|
davidwalsh
:
review+
imadueme
:
review+
|
Details | Review |
|
45 bytes,
text/x-phabricator-request
|
imadueme
:
review+
davidwalsh
:
review+
|
Details | Review |
|
45 bytes,
text/x-phabricator-request
|
imadueme
:
review+
|
Details | Review |
|
45 bytes,
text/x-phabricator-request
|
imadueme
:
review+
|
Details | Review |
The main authentication method for Lando API is auth0, this bug will track adding auth0 support.
|
||
Comment 1•8 years ago
|
||
Comment on attachment 8921585 [details]
auth0: add require_access_token decorator.
David Walsh :davidwalsh has approved the revision.
https://phabricator.services.mozilla.com/D155#3778
Attachment #8921585 -
Flags: review+
|
|
||
Comment 2•8 years ago
|
||
Comment on attachment 8921585 [details]
auth0: add require_access_token decorator.
Israel Madueme [:imadueme] has approved the revision.
https://phabricator.services.mozilla.com/D155#3810
Attachment #8921585 -
Flags: review+
|
|
||
Comment 3•8 years ago
|
||
Comment on attachment 8921586 [details]
auth0: add require_auth0_userinfo decorator.
Israel Madueme [:imadueme] has approved the revision.
https://phabricator.services.mozilla.com/D160#3862
Attachment #8921586 -
Flags: review+
|
|
||
Comment 4•8 years ago
|
||
Comment on attachment 8921586 [details]
auth0: add require_auth0_userinfo decorator.
David Walsh :davidwalsh has approved the revision.
https://phabricator.services.mozilla.com/D160#3870
Attachment #8921586 -
Flags: review+
|
|
||
Comment 5•8 years ago
|
||
Comment on attachment 8922980 [details]
auth0: add A0User class which checks group membership.
Israel Madueme [:imadueme] has approved the revision.
https://phabricator.services.mozilla.com/D170#4676
Attachment #8922980 -
Flags: review+
|
|
||
Comment 6•8 years ago
|
||
Comment on attachment 8925537 [details]
landings: require active_scm_level_3 to land.
Israel Madueme [:imadueme] has approved the revision.
https://phabricator.services.mozilla.com/D196#5062
Attachment #8925537 -
Flags: review+
|
Assignee | |
Comment 7•8 years ago
|
||
I believe the 4 changes above take care of "auth0 support" for lando-api. In the future we'll need to protect more endpoints, but all the pieces are there to make that easy, and POSTing to landings is protected, requiring active_scm_level_3.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•