Add comment to clarify the check for nonce URL

itsmeichigo · itsmeichigo · commit a175e8640db2 · 2025-10-22T07:34:02.000+07:00
diff --git a/WooCommerce/Classes/Authentication/SiteCredentialLoginUseCase.swift b/WooCommerce/Classes/Authentication/SiteCredentialLoginUseCase.swift
@@ -160,6 +160,9 @@ private extension SiteCredentialLoginUseCase {
             throw SiteCredentialLoginError.invalidLoginResponse
         }
 
+        /// The login request comes with a redirect header to nonce retrieval URL.
+        /// If we get a response from this URL, that means the redirect is successful.
+        /// We need to check the result of this redirect first to determine if login is successful.
         let isNonceUrl = response.url?.absoluteString.hasSuffix(Constants.wporgNoncePath) == true
 
         switch (isNonceUrl, response.statusCode) {

Original file line number	Diff line number	Diff line change
`@@ -160,6 +160,9 @@ private extension SiteCredentialLoginUseCase {`
`160`	`160`	`throw SiteCredentialLoginError.invalidLoginResponse`
`161`	`161`	`}`
`162`	`162`
	`163`	`+ /// The login request comes with a redirect header to nonce retrieval URL.`
	`164`	`+ /// If we get a response from this URL, that means the redirect is successful.`
	`165`	`+ /// We need to check the result of this redirect first to determine if login is successful.`
`163`	`166`	`let isNonceUrl = response.url?.absoluteString.hasSuffix(Constants.wporgNoncePath) == true`
`164`	`167`
`165`	`168`	`switch (isNonceUrl, response.statusCode) {`