twilio / authy-devise

Authy Devise plugin to add Two-Factor Authentication
MIT License
200 stars 84 forks source link

Record Authy Authentication After Successfully Verifying Installation #87

Closed nukturnal closed 6 years ago

nukturnal commented 6 years ago

The POST_verify_authy_installation after successful verification does not invoke record_authy_authentication method so some resources are not updated after very first time 2FA setup such as the last_sign_in_with_authy attribute on the model and the session["#{resource_name}_authy_token_checked"] session key.

This prevents one from using the session["user_authy_token_checked"] session key to check if user has been successfully verified/logged in with 2FA first-time.

PR to Fix for Issue #86

philnash commented 6 years ago

Thanks so much for this fix. And sorry it's taken ages to get merged.

It'll be part of the next release, which I will hopefully get out soon.