Feature/sign in with self hosted credentials master#1542
Conversation
…essage when site is not found
…irect to magic link screen
…f the xmlrpc url is empty
…to LoginNoJetpackFragment
…if site is wp.com
…sted-credentials Jetpack sign in - Step 1: Fetch Jetpack e-mail address for flowing through Magic Links
…e/1482-step2-magic-link-signin-changes
…github.com/woocommerce/woocommerce-android into issue/1482-step2-magic-link-signin-changes
Issue/1482 step4 track events
Generated by 🚫 dangerJS |
|
You can test the changes on this Pull Request by downloading the APK here. |
AmandaRiu
left a comment
There was a problem hiding this comment.
@anitaa1990 I ran through all the test scenarios and most of them were successful except for the following:
Scenario II: Login to a site with SSL Certificate required error:
I get the “Jetpack Required” screen even though I have jetpack installed and connected. If I bypass that screen and then login with the site credentials then I get the expected SSL Certificate connection error.
Login to a site with HTTP AUTH required error
When I click “try again” in step #9, it brings me back to the “login with credentials” screen. When I hit next, I get the “jetpack required” screen again…no snackbar error message. The rest of the steps work fine. NOTE that I though the way it worked was expected, I don't recall a snackbar for this scenario.
|
Thanks for testing @AmandaRiu!
Debugging further using the Jetpack Debugger, It looks like the test to check if jetpack features are available for the site results in an error. So the I tested again now and it looks like the issue is fixed on the site and it's working as expected. My guess is that something weird is going on in that site 🤦♀
So there are two scenarios where the user can be redirected to the
For the first option, when user clicks on For the second option, when user clicks on I found that in order for this reproduce this discovery error, you would need to enable the HTTP AUTH for the entire site.
Now if you follow the steps for |
A follow up for this issue @AmandaRiu. I created another test site: anitaastestwpsite3.com and was able to successfully simulate the ssl certificate error and the I believe there is some issue with http://anitaastestwpsite2.com/ that caused Jetpack to go undetected even when it was available 🤷♀ |
AmandaRiu
left a comment
There was a problem hiding this comment.
@anitaa1990 Thank you for the detailed explanation about when I would see the "Jetpack not installed" snack message. I was able to successfully re-test both scenarios and they passed with flying colors! Awesome work on this project! ![]()


Fixes #1482. This is the master branch for the sign in with self hosted credentials feature.
Screenshots
Site credentials login - with Jetpack
Site credentials login - without Jetpack
(LEFT: Try Login without jetpack . RIGHT: Activate Jetpack while on the app and click onTRY AGAIN)
. 
Discovery errors:
Logging into a site with HTTP AUTH required when Jetpack is NOT available:
(LEFT: When jetpack is still not installed and user clicks on Try AGAIN button)
. 
(RIGHT: When jetpack is installed when user is on the screen and clicks on TRY AGAIN button)
Logging into a site with XMLRPC missing when Jetpack is available:
Logging into a site with missing rsd tags:
Logging into a site with SSL Certificate error :
Logging into a site with the XML-RPC request is blocked :
Different discovery error messages:
(LEFT: HTTP AUTH required . RIGHT: SSL certificate needed)
. 
(LEFT: XMLRPC errors . RIGHT: All other generic errors)
. 
Testing
Test Current Functionality
Test New Functionality
Other Testing
Discovery error scenarios
Scenario I: Login to a site with XMLRPC file renamed.
wp-admin.Next.hasJetpack: true,isJetpackActive: false,isJetpackConnected: false.Login with site credentialsand enter username + password.We were unable to access the XMLRPC file on your site. You will need to reach out to your host to resolve this..Try againredirects to the Username + password screen.Read our troubleshooting tipsredirects to a webview from this link.Helpopens the help page.Sign in with Wordpress.comredirects to the Email screen and you are able to login successfully to the app using wp.com email.Scenario II: Login to a site with SSL Certificate required error:
(This scenario is the same as the above scenario but with different discovery error type:
ERRONEOUS_SSL_CERTIFICATE)wp-admin.Next.Login with site credentialsand enter username + password.We were unable to access your site because of a problem with the SSL Certificate. You will need to reach out to your host to resolve this..Try againredirects to the Username + password screen.Read our troubleshooting tipsredirects to a webview from this link.Helpopens the help page.Sign in with Wordpress.comredirects to the Email screen and you are able to login successfully to the app using wp.com email.Login to a site with HTTP AUTH required error
wp-admin.HTTP Authplugin from the same page. This enables HTTP Auth to the site.Next.Login with site credentialsand enter username + password.wp-adminand deactivateHTTP Authbefore activating and connecting Jetpack (This is because Jetpack does not support HTTP Auth plugin and you will only be able to connect Jetpack if HTTP Auth plugin is deactivated).Nextand verify that you are successfully redirected to the Magic link screen.Login to a site with XMLRPC is blocked.
Disable XML-RPC.We were unable to access the XMLRPC file on your site. You will need to reach out to your host to resolve this..Notes:
Update release notes:
RELEASE-NOTES.txtif necessary.