Nits

daffl · daffl · commit 3fd26b5e6077 · 2026-03-21T10:26:33.000-07:00
diff --git a/packages/authentication-oauth/src/strategy.ts b/packages/authentication-oauth/src/strategy.ts
@@ -144,11 +144,11 @@ export class OAuthStrategy extends AuthenticationBaseStrategy {
       return null
     }
 
-    const redirectUrl = `${redirect}${queryRedirect}`
-
     if (queryRedirect) {
       validateRedirectOrigin(redirect, queryRedirect)
     }
+
+    const redirectUrl = `${redirect}${queryRedirect}`
     const separator = redirectUrl.endsWith('?') ? '' : redirect.indexOf('#') !== -1 ? '?' : '#'
     const authResult: AuthenticationResult = data
     const query = authResult.accessToken

Original file line number	Diff line number	Diff line change
`@@ -144,11 +144,11 @@ export class OAuthStrategy extends AuthenticationBaseStrategy {`
`144`	`144`	`return null`
`145`	`145`	`}`
`146`	`146`
`147`		- const redirectUrl = `${redirect}${queryRedirect}`
`148`		`-`
`149`	`147`	`if (queryRedirect) {`
`150`	`148`	`validateRedirectOrigin(redirect, queryRedirect)`
`151`	`149`	`}`
	`150`	`+`
	`151`	+ const redirectUrl = `${redirect}${queryRedirect}`
`152`	`152`	`const separator = redirectUrl.endsWith('?') ? '' : redirect.indexOf('#') !== -1 ? '?' : '#'`
`153`	`153`	`const authResult: AuthenticationResult = data`
`154`	`154`	`const query = authResult.accessToken`