Page MenuHomePhabricator

Allow users to register with non-registration providers if they are invited to an instance
ClosedPublic

Authored by epriestley on Feb 7 2019, 2:04 AM.
Tags
None
Referenced Files
F18747070: D20118.id.diff
Fri, Oct 3, 2:32 PM
F18741611: D20118.diff
Thu, Oct 2, 2:49 PM
F18715597: D20118.diff
Mon, Sep 29, 10:30 AM
F18702558: D20118.id48033.diff
Sep 28 2025, 12:37 AM
F18518047: D20118.diff
Sep 5 2025, 2:23 PM
F18458518: D20118.id.diff
Sep 1 2025, 2:43 PM
F18439659: D20118.diff
Aug 31 2025, 3:17 PM
F18085181: D20118.diff
Aug 5 2025, 7:02 PM
Subscribers
None

Details

Summary

Depends on D20117. Fixes T10071. When you're sent an email invitation, it's intended to allow you to register an account even if you otherwise could not (see D11737).

Some time between D11737 and today, this stopped working (or perhaps it never worked and I got things wrong in D11737). I think this actually ended up not mattering for us, given the way Phacility auth was ultimately built.

This feature generally seems reasonable, though, and probably should work. Make it work in the "password" and "oauth" cases, at least. This may still not work for LDAP, but testing that is nontrivial.

Test Plan
  • Enabled only passwords, turned off registration, sent an invite, registered with a password.
  • Enabled only Google OAuth, turned off registration, sent an invite, registered with Google OAuth.

Diff Detail

Repository
rP Phabricator
Branch
xacc8
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21873
Build 29855: Run Core Tests
Build 29854: arc lint + arc unit