fix(auth): remove redundant validateCallbackUrl re-check on already-safe callbackUrl

waleedlatif1 · waleedlatif1 · commit 551664219b8e · 2026-03-19T13:55:46.000-07:00
diff --git a/apps/sim/app/(auth)/login/login-form.tsx b/apps/sim/app/(auth)/login/login-form.tsx
@@ -175,7 +175,7 @@ export default function LoginPage({
     }
 
     try {
-      const safeCallbackUrl = validateCallbackUrl(callbackUrl) ? callbackUrl : '/workspace'
+      const safeCallbackUrl = callbackUrl
       let errorHandled = false
 
       const result = await client.signIn.email(
diff --git a/apps/sim/ee/sso/components/sso-form.tsx b/apps/sim/ee/sso/components/sso-form.tsx
@@ -98,7 +98,7 @@ export default function SSOForm() {
     }
 
     try {
-      const safeCallbackUrl = validateCallbackUrl(callbackUrl) ? callbackUrl : '/workspace'
+      const safeCallbackUrl = callbackUrl
 
       await client.signIn.sso({
         email: emailValue,

Original file line number	Diff line number	Diff line change
`@@ -175,7 +175,7 @@ export default function LoginPage({`
`175`	`175`	`}`
`176`	`176`
`177`	`177`	`try {`
`178`		`- const safeCallbackUrl = validateCallbackUrl(callbackUrl) ? callbackUrl : '/workspace'`
	`178`	`+ const safeCallbackUrl = callbackUrl`
`179`	`179`	`let errorHandled = false`
`180`	`180`
`181`	`181`	`const result = await client.signIn.email(`
Original file line number	Diff line number	Diff line change
`@@ -98,7 +98,7 @@ export default function SSOForm() {`
`98`	`98`	`}`
`99`	`99`
`100`	`100`	`try {`
`101`		`- const safeCallbackUrl = validateCallbackUrl(callbackUrl) ? callbackUrl : '/workspace'`
	`101`	`+ const safeCallbackUrl = callbackUrl`
`102`	`102`
`103`	`103`	`await client.signIn.sso({`
`104`	`104`	`email: emailValue,`