Page MenuHomePhabricator

Don't require users be activated in order to establish Web sessions
ClosedPublic

Authored by epriestley on Jun 10 2015, 8:24 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Jun 6, 3:18 AM
Unknown Object (File)
Mon, May 26, 2:29 AM
Unknown Object (File)
Sun, May 25, 3:44 PM
Unknown Object (File)
Sat, May 24, 9:06 PM
Unknown Object (File)
Apr 29 2025, 12:59 AM
Unknown Object (File)
Apr 20 2025, 11:47 PM
Unknown Object (File)
Apr 19 2025, 11:52 PM
Unknown Object (File)
Apr 19 2025, 1:14 PM

Details

Summary

Ref T8496. In D13123, the condition for establishing a web session was made too strict: we need to let non-activated users establish web sessions in order to see "you are a bad disabled person" or "your account needs approval" messages. The previous behavior let them in, the new behavior incorrectly locks them out.

Test Plan

Enabled login approvals and registered a new account with username/password auth.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable