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)
Thu, Aug 11, 6:38 AM
Unknown Object (File)
Fri, Aug 5, 7:00 PM
Unknown Object (File)
Fri, Aug 5, 1:14 PM
Unknown Object (File)
Jul 13 2022, 4:38 AM
Unknown Object (File)
Jun 27 2022, 6:34 AM
Unknown Object (File)
Jun 17 2022, 1:16 AM
Unknown Object (File)
May 27 2022, 1:27 PM
Unknown Object (File)
Jan 29 2017, 4:04 AM

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