From 528fbbce4ab02c0d21102ae98d0a1d45156a63bd Mon Sep 17 00:00:00 2001
From: Michael Zetterberg <michael.zetterberg@scandichotels.com>
Date: Tue, 27 May 2025 13:39:30 +0200
Subject: [PATCH] fix: change to permanent redirects were applicable

---
 apps/scandic-web/middlewares/dateQueryParams.ts    | 4 +++-
 apps/scandic-web/middlewares/legacySearchParams.ts | 4 +++-
 apps/scandic-web/middlewares/myPages.ts            | 4 +++-
 apps/scandic-web/middlewares/redirect.ts           | 3 ++-
 apps/scandic-web/middlewares/trailingSlash.ts      | 1 +
 5 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/apps/scandic-web/middlewares/dateQueryParams.ts b/apps/scandic-web/middlewares/dateQueryParams.ts
index bde91b2ce..1adf0418c 100644
--- a/apps/scandic-web/middlewares/dateQueryParams.ts
+++ b/apps/scandic-web/middlewares/dateQueryParams.ts
@@ -26,7 +26,9 @@ export const middleware: NextMiddleware = (request) => {
 
   if (modified) {
     url.search = searchParams.toString()
-    return NextResponse.redirect(url)
+    return NextResponse.redirect(url, {
+      status: 308,
+    })
   }
 
   return NextResponse.next()
diff --git a/apps/scandic-web/middlewares/legacySearchParams.ts b/apps/scandic-web/middlewares/legacySearchParams.ts
index 7849a2a81..52d18186d 100644
--- a/apps/scandic-web/middlewares/legacySearchParams.ts
+++ b/apps/scandic-web/middlewares/legacySearchParams.ts
@@ -50,7 +50,9 @@ export const middleware: NextMiddleware = (request) => {
     url.searchParams.set("todate", normalizedToDate)
   }
 
-  return NextResponse.redirect(url)
+  return NextResponse.redirect(url, {
+    status: 308,
+  })
 }
 
 export const matcher: MiddlewareMatcher = (request) => {
diff --git a/apps/scandic-web/middlewares/myPages.ts b/apps/scandic-web/middlewares/myPages.ts
index cbf23b1e0..21da23fc8 100644
--- a/apps/scandic-web/middlewares/myPages.ts
+++ b/apps/scandic-web/middlewares/myPages.ts
@@ -26,7 +26,9 @@ export const middleware: NextMiddleware = async (request) => {
     const overviewUrl = overview[lang]
     const redirectUrl = new URL(overviewUrl, nextUrlPublic)
     console.log(`[myPages] redirecting to: ${redirectUrl}`)
-    return NextResponse.redirect(redirectUrl)
+    return NextResponse.redirect(redirectUrl, {
+      status: 308,
+    })
   }
 
   const pathNameWithoutLang = nextUrl.pathname.replace(`/${lang}`, "")
diff --git a/apps/scandic-web/middlewares/redirect.ts b/apps/scandic-web/middlewares/redirect.ts
index 01a5511a5..7b99d7a50 100644
--- a/apps/scandic-web/middlewares/redirect.ts
+++ b/apps/scandic-web/middlewares/redirect.ts
@@ -54,9 +54,10 @@ export const middleware: NextMiddleware = async (request) => {
 
     if (matchedRedirect) {
       const newUrl = new URL(matchedRedirect, getPublicNextURL(request))
-      headers.set("Cache-control", "public, max-age=60")
+      headers.set("Cache-control", "public, max-age=14400") // 4 hours
       return NextResponse.redirect(newUrl, {
         headers,
+        status: 308,
       })
     }
     headers.set("x-continue", "1")
diff --git a/apps/scandic-web/middlewares/trailingSlash.ts b/apps/scandic-web/middlewares/trailingSlash.ts
index 5bb2fb7cd..affa31cf8 100644
--- a/apps/scandic-web/middlewares/trailingSlash.ts
+++ b/apps/scandic-web/middlewares/trailingSlash.ts
@@ -16,6 +16,7 @@ export const middleware: NextMiddleware = async (request) => {
 
   return NextResponse.redirect(newUrl, {
     headers,
+    status: 308,
   })
 }