refactor, comments

src/api/apiAxios.ts

@@ -29,6 +29,8 @@ export const newApiAxios = (base: 'server' | 'client') => {
 
       debugServer('[SERVER API]', method?.toUpperCase(), url && baseURL ? new URL(url, baseURL).href : url)
 
+      // server-side requesty z deployed FE na deployed BE potrebuju tieto hlavicky
+      // TODO: ked pojdeme do produkcie, asi bude treba riesit nejakym env varom
       config.headers['X-Forwarded-Host'] = 'test.strom.sk'
       config.headers['X-Forwarded-Proto'] = 'https'
 

src/api/apiInterceptor.ts

@@ -1,14 +1,14 @@
 import axios from 'axios'
 
-import {addApiTrailingSlash} from '@/utils/addApiTrailingSlash'
+import {addTrailingSlash} from '@/utils/trailingSlash'
 
 type RequestInterceptor = Parameters<typeof axios.interceptors.request.use>[0]
 
 export const apiInterceptor: RequestInterceptor = (config) => {
   if (config.url) {
     const [pathname, query] = config.url.split('?')
 
-    const newPathname = addApiTrailingSlash(pathname)
+    const newPathname = addTrailingSlash(pathname)
 
     config.url = `${newPathname}${query ? `?${query}` : ''}`
   }

src/middleware.ts

@@ -1,6 +1,7 @@
 import {NextRequest, NextResponse} from 'next/server'
 
 import {backendRewriteMiddleware} from './middleware/backendRewriteMiddleware'
+import {removeTrailingSlash} from './utils/trailingSlash'
 
 export function middleware(req: NextRequest) {
   const url = req.nextUrl
@@ -16,9 +17,10 @@ export function middleware(req: NextRequest) {
   // odstran trailing slash - default next.js spravanie, ale vypli sme ho v next.config.ts pomocou
   // `skipTrailingSlashRedirect: true`, aby sme dovolili (a v axiose a middlewari vyssie aj forcli)
   // trailing slash pre BE Django
-  // pre root route `/` to nemozeme spravit (infinite redirect) ¯\_(ツ)_/¯
-  if (url.pathname.endsWith('/') && url.pathname !== '/') {
-    const newPathname = url.pathname.slice(0, -1)
+  const newPathname = removeTrailingSlash(url.pathname)
+
+  // redirect ak sa da odstranit trailing slash
+  if (newPathname !== url.pathname) {
     const newUrl = new URL(`${newPathname}${url.search}`, url)
 
     return NextResponse.redirect(newUrl, 308)

src/middleware/backendRewriteMiddleware.ts

@@ -1,6 +1,6 @@
 import {NextRequest, NextResponse} from 'next/server'
 
-import {addApiTrailingSlash} from '@/utils/addApiTrailingSlash'
+import {addTrailingSlash} from '@/utils/trailingSlash'
 import {getBackendServerUrl} from '@/utils/urlBase'
 
 export const backendRewriteMiddleware = ({req, trailingSlash}: {req: NextRequest; trailingSlash: boolean}) => {
@@ -9,7 +9,7 @@ export const backendRewriteMiddleware = ({req, trailingSlash}: {req: NextRequest
 
   let newPathname = pathname
 
-  if (trailingSlash) newPathname = addApiTrailingSlash(newPathname)
+  if (trailingSlash) newPathname = addTrailingSlash(newPathname)
 
   const newUrl = new URL(`${newPathname}${search}`, getBackendServerUrl())
 

src/utils/trailingSlash.ts

@@ -0,0 +1,20 @@
+/**
+ * BE Django server ocakava trailing slash (aj pred query params).
+ *
+ * @param pathname local URL without query params. e.g. `/cms/post`
+ * @returns local URL with trailing slash. e.g. `/cms/post/`
+ *  */
+export const addTrailingSlash = (pathname: string) => {
+  if (!pathname.endsWith('/')) return `${pathname}/`
+
+  return pathname
+}
+
+export const removeTrailingSlash = (pathname: string) => {
+  // pre root route `/` nemozeme odstranit slash (infinite redirect)
+  if (pathname === '/') return pathname
+
+  if (pathname.endsWith('/')) return pathname.slice(0, -1)
+
+  return pathname
+}