Ref T3472. Ref T12113. This implements the gigantic roadblock nonsense in T3472.
Details
Details
- Reviewers
chad - Maniphest Tasks
- Restricted Maniphest Task
T12113: Email in LDAP registration form is empty - Commits
- rP23721799fd0b: Explicitly warn the user multiple times when they try to register an external…
Diff Detail
Diff Detail
- Repository
- rP Phabricator
- Branch
- regwarn1
- Lint
Lint Passed - Unit
Tests Passed - Build Status
Buildable 15288 Build 20117: Run Core Tests Build 20116: arc lint + arc unit
Event Timeline
Comment Actions
We could possibly improve this flow by giving users a modal choice ("link existing" or "create new") instead of this roadblock dialog ("stop, go do it right" or "create new"), but we'd need to put more handholding along the "link existing" flow because the user must login (and go through MFA, if applicable) before we can safely link accounts: it is not safe to link accounts just because a user knows the credential of some external account with the same email address.
This flow is rare anyway, and this roadblock mostly saves us from spending 2+ hours failing to reproduce LDAP issues in the future (T12113).
Comment Actions
Just for the record, this change works for me. Many thanks for quickly resolving my problem!