Page MenuHomePhabricator

Allow login to be disabled for authentication providers
ClosedPublic

Authored by epriestley on Dec 15 2015, 10:52 PM.
Tags
None
Referenced Files
F14352758: D14794.diff
Thu, Dec 19, 2:43 PM
Unknown Object (File)
Thu, Dec 12, 2:37 PM
Unknown Object (File)
Sat, Dec 7, 11:19 AM
Unknown Object (File)
Fri, Dec 6, 9:56 AM
Unknown Object (File)
Sun, Dec 1, 7:12 AM
Unknown Object (File)
Thu, Nov 28, 6:30 PM
Unknown Object (File)
Thu, Nov 28, 6:30 PM
Unknown Object (File)
Thu, Nov 28, 6:27 PM
Subscribers
None

Details

Summary

Fixes T9997. This was in the database since v0, I just never hooked up the UI since it wasn't previously meaningful.

However, it now makes sense to have a provider like Asana with login disabled and use it only for integrations.

Test Plan

Disabled login on a provider, verified it was no longer available for login/registration but still linkable.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Allow login to be disabled for authentication providers.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

Specifically, per T9997, a use case for this is:

  • You want to require login through Google, or some other provider with strong identity guarantees (e.g., mandatory MFA).
  • You also want integrations with an external OAuth system like Asana or JIRA, or other systems in the future.

This is incidentally convenient for us as Phacility since we want instance users to login through central auth, but the primary use case is a reasonable one in geneal.

chad edited edge metadata.
This revision is now accepted and ready to land.Dec 15 2015, 10:58 PM
This revision was automatically updated to reflect the committed changes.