Setting destination query param the right way

eozden-wq · Lordfirespeed · commit f5719feac396 · 2025-10-15T21:09:15.000+01:00
diff --git a/client/src/middleware.ts b/client/src/middleware.ts
@@ -25,7 +25,8 @@ async function getUserProfile(request: NextRequest): Promise<User | null> {
 }
 
 function redirectUnauthenticated(request: NextRequest) {
-  const loginUrl = new URL(`/auth/keycloak/login?destination=${request.url}`, siteConfig.apiUrl)
+  const loginUrl = new URL("/auth/keycloak/login", siteConfig.apiUrl)
+  loginUrl.searchParams.set("destination", request.url)
   return NextResponse.redirect(loginUrl)
 }
 

Original file line number	Diff line number	Diff line change
`@@ -25,7 +25,8 @@ async function getUserProfile(request: NextRequest): Promise<User \| null> {`
`25`	`25`	`}`
`26`	`26`
`27`	`27`	`function redirectUnauthenticated(request: NextRequest) {`
`28`		- const loginUrl = new URL(`/auth/keycloak/login?destination=${request.url}`, siteConfig.apiUrl)
	`28`	`+ const loginUrl = new URL("/auth/keycloak/login", siteConfig.apiUrl)`
	`29`	`+ loginUrl.searchParams.set("destination", request.url)`
`29`	`30`	`return NextResponse.redirect(loginUrl)`
`30`	`31`	`}`
`31`	`32`