Wordpress-ios: [GlobalStep] Two Factor Authentication Screen is displayed as an Authenticator App is being used even when the method of choice is Text Messages.

Created on 24 Apr 2020  Â·  6Comments  Â·  Source: wordpress-mobile/WordPress-iOS

Description

Applying Two Factor Authentication to an account and choosing Text Messages as the Authentication method before attempting to Login causes the message “Almost there! Please enter the verification code from your authenticator app.” and the button that reads “Text me a code instead” to be displayed, even though an Authenticator App is not being used and the text code was already sent.

Reproduction Rate

4/4 100%

Expected behaviour

The Two Factor Authentication screen should accurately reflect the Authentication method of choice.

Actual behaviour

The Two Factor Authentication screen is displayed as an Authenticator App is being used.

Steps to reproduce the behaviour

  1. Install and launch WordPress 14.7
  2. Activate Two Factor Authentication for an account and select Text Messages as the Authentication method.
  3. Enter the credentials for that account to reach the Two Factor Authentication Screen.
    Tested on the following

iPhone 7+ (12.0.1)

Please see the attached screenshot for more information

Submitted by:

Luis Pimenta

Login [Type] Bug

Most helpful comment

Is the login and signup going to be overhauled?

Yes.

So that means this bug can be ignored?

I'd rather not just close it. It's not explicitly pointed out in the new design. So I'll add it to the project to verify we do address it. We'll close it at that time.

(cc @mindgraffiti )

All 6 comments

Tested and confirmed using WPiOS 14.7.0.2 TestFlight beta.

Screen Shot 2020-04-24 at Apr 24 3 19 54 PM IMG_3930 IMG_3931

Tested with WPiOS 14.7.0.2 TestFlight beta on iPhone 11 iOS 13.4.1.

Inquired about addressing as part of the Unified Login & Signup project.

(internal reference: pauD4L-Bb-p2#comment-639)

Is the login and signup going to be overhauled? So that means this bug can be ignored?

Is the login and signup going to be overhauled?

Yes.

So that means this bug can be ignored?

I'd rather not just close it. It's not explicitly pointed out in the new design. So I'll add it to the project to verify we do address it. We'll close it at that time.

(cc @mindgraffiti )

I did some research to see if we could detect what type of 2FA an account has so we can customize the display text. Right now, we cannot.

  • We don't have any endpoints implemented for WP login that provides the 2FA type.
  • The endpoint we are using simply fails with a generic message that a 2FA code is needed. (/oauth2/token)

The social login endpoints do return the 2FA types (stored in SocialLogin2FANonceInfo). We _might_ be able to use the WP login equivalent (/wp-login.php). WPiOS does use this endpoint. However, there is this note in Blog.h: Can we use an OAuth2 token with wp-login.php?. That seems significant...

At this point, I'm not sure if there is an endpoint we can use to just query the 2FA types. Or if we can/should change the login endpoint. More research is required. This is out of scope for the ULS project, so I'm removing it. I can look at possibly improving/clarifying the messaging, but the ULS 2FA behavior will be the same as it is now.

cc @mindgraffiti

Was this page helpful?
0 / 5 - 0 ratings

Related issues

rachelmcr picture rachelmcr  Â·  21Comments

iamgabrielma picture iamgabrielma  Â·  90Comments

diegoreymendez picture diegoreymendez  Â·  20Comments

sendhil picture sendhil  Â·  29Comments

sentry-io[bot] picture sentry-io[bot]  Â·  19Comments