Closed
Bug 1404461
Opened 7 years ago
Closed 7 years ago
Remove all UI from login site
Categories
(Taskcluster :: Services, enhancement)
Taskcluster
Services
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: dustin, Assigned: dustin)
References
Details
This can occur once:
- Treeherder is upgraded (bug 1395356)
- Releng services are upgraded (bug 1395357)
- CLI logins are supported (bug 1395349) (this will cover logins from tools development too)
The root (https://login.taskcluster.net/) will redirect to whatever endpoint we devise in bug 1395349.
Assignee | ||
Comment 1•7 years ago
|
||
https://github.com/taskcluster/taskcluster-login/pull/67 removes some of the UI :)
Assignee | ||
Comment 2•7 years ago
|
||
requested [change-freeze]
Assignee | ||
Comment 3•7 years ago
|
||
response to the change-freeze from ed was
---
Treeherder supports logging in via the email option (for parity with the contributor experience we had with Persona), so this change would mean people could no longer do so.
Querying the prod Treeherder DB now, I see 474 users using LDAP and 71 using email. Filtering to those who have logged in during the last 30 days, there are 278 and 25 respectively. The users logging in via email appear to be a combination of (a) contributors (who may/may not have an LDAP account), (b) employees using non-moco or moco-alias email accounts.
The employee non-canonical email address use-case isn't one I feel needs supporting. The contributor one is possibly valid - in that not all contributors have an LDAP account, but may still want to assist with sheriffing (which requires being logged in) - though I'd be fine with dropping support in the future if everyone else was fine with it.
Best wishes,
Ed
---
so we held off.
Assignee | ||
Comment 5•7 years ago
|
||
Comment 6•7 years ago
|
||
Commits pushed to master at https://github.com/taskcluster/taskcluster-login
https://github.com/taskcluster/taskcluster-login/commit/3dcaad9ccbbcfcad134773b1e89f508fce0d5926
Bug 1404461 - remove most of the jade UI
..and the express configuration for it, including cookies, passport,
etc.
https://github.com/taskcluster/taskcluster-login/commit/7b7b8bec66d6285d7e8c513b6223792f0ff5fcec
Bug 1404461 - upgrade taskcluster-* dependencies
https://github.com/taskcluster/taskcluster-login/commit/51fcc28acce669b7699c9ad98aa0c9cdf572a466
Bug 1404461 - remove authenticators
https://github.com/taskcluster/taskcluster-login/commit/e312f5f3ef37cf5685d54c3833350e8549b59203
Merge pull request #76 from djmitche/bug1404461
Bug 1404461 - delete UI from login site
Assignee | ||
Comment 7•7 years ago
|
||
Wooooo!!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Component: Login → Services
You need to log in
before you can comment on or make changes to this bug.
Description
•