wordpress-mobile / WordPress-Login-Flow-Android

Pluggable WordPress login flow for Android
GNU General Public License v2.0
14 stars 3 forks source link

Strip know paths before sending the site-info API request #110

Closed hichamboushaba closed 1 year ago

hichamboushaba commented 1 year ago

When reviewing https://github.com/woocommerce/woocommerce-android/pull/8323, @JorgeMucientes identified an issue where the WPCom API /site-info fails to identify Jetpack sites if the path has some suffixes like wp-admin.

This PR fixes this by stripping known paths before sending the API request.

JorgeMucientes commented 1 year ago

Code looks good. Thanks for working on this @hichamboushaba! I'll merge asap as soon as I finished e2e from this PR