Page MenuHomePhabricator

Roadblock users trying to register with external accounts that have invalid emails
ClosedPublic

Authored by epriestley on Nov 12 2013, 7:58 PM.
Tags
None
Referenced Files
F14635698: D7571.diff
Fri, Jan 10, 10:16 PM
Unknown Object (File)
Thu, Jan 2, 1:43 PM
Unknown Object (File)
Tue, Dec 31, 9:06 PM
Unknown Object (File)
Fri, Dec 13, 7:53 AM
Unknown Object (File)
Dec 7 2024, 11:57 AM
Unknown Object (File)
Dec 6 2024, 10:57 AM
Unknown Object (File)
Nov 26 2024, 7:20 PM
Unknown Object (File)
Nov 19 2024, 6:22 PM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
Restricted Maniphest Task
Commits
Restricted Diffusion Commit
rPcd73fe78db36: Roadblock users trying to register with external accounts that have invalid…
Summary

Ref T3472. Currently, if an install only allows "@mycompany.com" emails and you try to register with an "@personal.com" account, we let you pick an "@mycompany.com" address instead. This is secure: you still have to verify the email. However, it defies user expectation -- it's somewhat confusing that we let you register. Instead, provide a hard roadblock.

(These accounts can still be linked, just not used for registration.)

Test Plan

See screenshot.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

src/applications/auth/controller/PhabricatorAuthRegisterController.php
71–73

Oh, maybe something that you can link the account once you register?

src/applications/auth/controller/PhabricatorAuthRegisterController.php
71–73

I worry that users have a tenuous understanding of what "linking accounts" means, and am hesitant to put more words here than necessary. If people show up with support questions about this, that would indicate to me that it might be useful to provide more details, though.