Skip to content
This repository has been archived by the owner on May 30, 2022. It is now read-only.

API user deletion on disable. #144

Merged
merged 1 commit into from
Jun 4, 2020
Merged

Conversation

philnash
Copy link
Contributor

@philnash philnash commented Jun 2, 2020

When a user disables 2FA, don't delete the user from the API if another user share's their authy_id.

Fixes #143.

Contributing to Twilio

All third-party contributors acknowledge that any contributions they provide will be made under the same open-source license that the open-source project is provided under.

  • I acknowledge that all my contributions will be made under the project's license.

When a user disables 2FA, don't delete the user from the API if another user share's their authy_id. Fixes twilio#143.
@dpflucas
Copy link

dpflucas commented Jun 2, 2020

LGTM, thanks!

@philnash philnash merged commit 03bebb5 into twilio:master Jun 4, 2020
@philnash philnash deleted the user-deletion branch June 4, 2020 00:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Option to disable 2FA without removing the User from Authy
2 participants