@W-20935841: Bug - P2 - [Android] Login Server Switching Misbehaving

[JohnsonEricAtSalesforce]

🥁 Ready For Review 🥁

Salesforce Welcome Discovery's "mobile flow" requires the mobile app to access https://welcome.salesforce.com/discovery with the /discovery path plus the query string parameters client_id, client_version and callback_url. An interesting and possibly not supported use case is if app is also configured with https://welcome.salesforce.com as a login server that does not have the /discovery path. This triggers a failed server switch where the authorization URL isn't loaded and the progress indicator keeps spinning.

The actual logic change is that up until WSC Disco path was not a consideration for the login server. In this bug, the login server doesn't update since MSDK cannot tell the two WSC URLs are different since they have the same host and prefix.

To resolve this, the URL check now examines the host and path discretely to drive the logic and doesn't use string level comparison for the URL values.

Here's a recording of the new behavior. Note the WSC entry without the required WSC /discovery suffix doesn't respect the consumer key validation and allows an app outside the allow list to access WSC. There's no client_id parameter in that case to drive the server-side logic. Currently, WSC hosts don't have a host naming convention to drive the submission of those parameters which is the purpose of the path. Only the production host has a fixed host name.
Screen_recording_20260120_130154.webm

[github-actions]

	1 Error
🚫	Please re-submit this PR to the dev branch, we may have already fixed your issue.

Generated by 🚫 Danger

[github-actions]

Job Summary for Gradle

Pull Request :: test-android

Gradle Root Project	Requested Tasks	Gradle Version	Build Outcome	Build Scan®
SalesforceMobileSDK-Android	libs:SalesforceSDK:convertCodeCoverage	8.14.2	✅
SalesforceMobileSDK-Android	libs:SalesforceSDK:lint	8.14.2	✅
SalesforceMobileSDK-Android	libs:SalesforceSDK:assembleAndroidTest	8.14.2	❌

[wmathurin]

Let's have another PR afterwards to bring the changes back from master to dev.