HomePhorge

Improve handling of "Deny" responses from Duo

Description

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, re-denied, tried to pass an MFA gate. Got another nice helpful error message.

Reviewers: amckinley

Reviewed By: amckinley

Maniphest Tasks: T13231

Differential Revision: https://secure.phabricator.com/D20065

Details

Provenance
epriestleyAuthored on Jan 29 2019, 8:45 PM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP93b512b63cca: Update a factor query in TransactionEditor for providers
Branches
Loading...
Tags
Loading...

Event Timeline