Fikser middleware, de skal ikke returnere responser

Author: hanneolsen

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/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;
 };