Page MenuHomePhabricator

Don't show "registration might be too open" warnings unless an auth provider actually allows registration
ClosedPublic

Authored by epriestley on Feb 7 2019, 2:14 AM.
Tags
None
Referenced Files
F15470538: D20119.id48034.diff
Fri, Apr 4, 11:19 PM
F15470537: D20119.id48063.diff
Fri, Apr 4, 11:19 PM
F15470447: D20119.diff
Fri, Apr 4, 10:19 PM
F15469715: D20119.diff
Fri, Apr 4, 3:50 PM
F15421140: D20119.id48063.diff
Fri, Mar 21, 9:15 PM
Unknown Object (File)
Feb 25 2025, 9:49 AM
Unknown Object (File)
Feb 20 2025, 12:55 PM
Unknown Object (File)
Feb 9 2025, 7:07 AM
Subscribers
None

Details

Summary

Depends on D20118. Fixes T5351. We possibly raise some warnings about registration (approval queue, email domains), but they aren't relevant if no one can register.

Hide these warnings if no providers actually support registration.

Test Plan

Viewed the Auth provider list with registration providers and with no registration providers, saw more tailored guidance.

Diff Detail

Repository
rP Phabricator
Branch
xacc9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 21874
Build 29857: Run Core Tests
Build 29856: arc lint + arc unit