R1:e9b2d667ee57
R1:e9b2d667ee57
Improve handling of "Deny" responses from Duo
Summary:
Ref T13231. See <https://discourse.phabricator-community.org/t/duo-integration-crashes-if-user-is-not-enrolled-and-enrollment-is-disabled/2340/5>
(There's an actual bug here, although I'm not sure exactly what's going on on the Duo side in the report.)
Test Plan:
To reproduce this, I was only able to actually "Deny" my account explicitly in Duo.
- With "Deny", tried to add a factor. Got a nice helpful error message.
- Undenied, added a factor…
Summary:
Ref T13231. See <https://discourse.phabricator-community.org/t/duo-integration-crashes-if-user-is-not-enrolled-and-enrollment-is-disabled/2340/5>
(There's an actual bug here, although I'm not sure exactly what's going on on the Duo side in the report.)
Test Plan:
To reproduce this, I was only able to actually "Deny" my account explicitly in Duo.
- With "Deny", tried to add a factor. Got a nice helpful error message.
- Undenied, added a factor…
Repository: R1 hydra
Commit Date: Jan 30 2019