Skip to content

Conversation

gbrl
Copy link

@gbrl gbrl commented Oct 28, 2016

No description provided.

gbrl added 3 commits October 28, 2016 12:17
I struggled getting this gem working until I realized that I had to use the github repo as my gem src. So I updated the README.md file accordingly. Please modify it as you see fit.
@maschwenk
Copy link

This fixes my issues running 1.1.4+ in Rails >= 5. Any plans for this to be merged?

if user.respond_to?(:need_two_factor_authentication?) && !bypass_by_cookie
if auth.session(options[:scope])[TwoFactorAuthentication::NEED_AUTHENTICATION] = user.need_two_factor_authentication?(auth.request)
user.send_new_otp unless user.totp_enabled?
user.send_new_otp

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why has this changed? What about Google Authenticator users?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you mean?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, misunderstanding. All good 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants