Bump express fra 4 til 5 (#1510)

* Oppgraderer express versjon fra 4.x til 5.x

* Oppdaterer splat route

* Oppdaterer familie-logging

* Oppdaterer dotenv

* Oppdaterer @types/express

* Fjerner ubrukt dependency @types/express-session

* Fikser middleware, de skal ikke returnere responser

Author: hanneolsen

package.json

@@ -85,7 +85,7 @@
     "@navikt/ds-css": "^6.1.3",
     "@navikt/ds-react": "^6.1.3",
     "@navikt/familie-form-elements": "^15.0.0",
-    "@navikt/familie-logging": "^7.0.0",
+    "@navikt/familie-logging": "^7.0.3",
     "@navikt/familie-skjema": "^8.0.8",
     "@navikt/familie-typer": "^8.0.2",
     "@navikt/fnrvalidator": "^2.0.7",
@@ -102,8 +102,8 @@
     "cors": "^2.8.5",
     "csp-header": "^6.0.0",
     "date-fns": "^4.1.0",
-    "dotenv": "^16.4.5",
-    "express": "^4.21.1",
+    "dotenv": "^16.4.7",
+    "express": "^5.0.1",
     "html-react-parser": "^5.2.2",
     "http-proxy-middleware": "^3.0.3",
     "i18n-iso-countries": "^7.11.0",
@@ -139,8 +139,7 @@
     "@types/cookie-parser": "^1.4.7",
     "@types/cors": "^2.8.17",
     "@types/eslint-plugin-jsx-a11y": "^6.10.0",
-    "@types/express": "^4.17.21",
-    "@types/express-session": "^1.18.0",
+    "@types/express": "^5.0.0",
     "@types/jest": "^29.5.11",
     "@types/jsonwebtoken": "^9.0.5",
     "@types/mustache-express": "^1.2.5",

src/backend/middlewares/erklaering-interceptor.ts

@@ -1,4 +1,4 @@
-import { RequestHandler, Request, Response } from 'express';
+import { Request, Response, NextFunction, RequestHandler } from 'express';
 
 import { byggFeiletRessurs } from '@navikt/familie-typer';
 
@@ -19,7 +19,7 @@ export const hentSpråkteksterAlleSpråk = (språknøkkel: string): Record<Local
 export const erklaeringInterceptor: RequestHandler = (
     request: Request,
     response: Response,
-    next
+    next: NextFunction
 ) => {
     const søknad: ISøknadKontrakt = request.body;
     const spmKey = 'lestOgForståttBekreftelse';
@@ -29,15 +29,16 @@ export const erklaeringInterceptor: RequestHandler = (
     if (
         !('spørsmål' in søknad && spmKey in søknad.spørsmål && 'verdi' in søknad.spørsmål[spmKey])
     ) {
-        return response.status(400).send(byggFeiletRessurs('Ugyldig søknadformat'));
+        response.status(400).send(byggFeiletRessurs('Ugyldig søknadformat'));
+        return;
     }
 
     const svar = søknad.spørsmål[spmKey];
 
     if (aksepterteSvar.includes(svar.verdi[søknad.originalSpråk])) {
         next();
     } else {
-        return response
+        response
             .status(403)
             .send(byggFeiletRessurs('Du må huke av for at du oppgir korrekte opplysninger'));
     }

src/backend/middlewares/tokenProxy.ts

@@ -26,7 +26,7 @@ const attachToken = (applicationName: ApplicationName): RequestHandler => {
                 LOG_LEVEL.WARNING,
                 error
             );
-            return res.status(401).send('En uventet feil oppstod. Ingen gyldig token');
+            res.status(401).send('En uventet feil oppstod. Ingen gyldig token');
         }
     };
 };

src/backend/routes/index.ts

@@ -17,7 +17,7 @@ export const konfigurerIndex = (app: Express): Express => {
 
 export const konfigurerIndexFallback = (app: Express): Express => {
     // Fallback, alt vi ikke treffer med andre handlere returnerer index.html
-    app.get('*', indexHandler);
+    app.get('*splat', indexHandler);
     return app;
 };
 

src/backend/routes/modellversjon.ts

@@ -6,6 +6,8 @@ import { basePath } from '../../shared-utils/Miljø';
 import { modellVersjon } from '../../shared-utils/modellversjon';
 
 export const konfigurerModellVersjonEndpoint = (app: Express): Express => {
-    app.get(`${basePath}modellversjon`, (_, res) => res.send(byggSuksessRessurs(modellVersjon)));
+    app.get(`${basePath}modellversjon`, (_, res) => {
+        res.send(byggSuksessRessurs(modellVersjon));
+    });
     return app;
 };

src/backend/routes/nais.ts

@@ -1,6 +1,8 @@
 import { Express } from 'express';
 
 export const konfigurerNais = (app: Express): Express => {
-    app.get(/^\/(internal\/)?(isAlive|isReady)\/?$/, (_req, res) => res.sendStatus(200));
+    app.get(/^\/(internal\/)?(isAlive|isReady)\/?$/, (_req, res) => {
+        res.sendStatus(200);
+    });
     return app;
 };