Sonar fixes

[gunnsteingarmo]

Hvorfor er denne endringen nødvendig? ✨

Sonar peker på to ting som kan utbedres:

• Add an initial value to this "reduce()" call.
• This image might run with root as the default user. Make sure it is safe here. https://sonarcloud.io/project/security_hotspots?id=navikt_familie-ef-sak-frontend&sinceLeakPeriod=true

Denne PRen skal utbedre disse.

[ViktorGSolberg]

Vil du forklare i PRen hva endringene i dockerfilen betyr? Hadde vært fint med en forklaring slik at man slipper å søke opp dette selv 😊

Ville også squashet denne 👍

[ViktorGSolberg]

behøves det å kopiere over alt, hvis det er det denne linjen betyr? Vi prøver å bare kopiere over de filene containeren har bruk for

[sonarcloud]

Quality Gate passed

Issues
1 New issue
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[gunnsteingarmo]

Vil du forklare i PRen hva endringene i dockerfilen betyr? Hadde vært fint med en forklaring slik at man slipper å søke opp dette selv 😊

Ville også squashet denne 👍

Med hjelp fra copilot 🙏
Separasjon av bygge- og kjøretidsmiljøer: Byggestadiet (FROM node:18-alpine as builder) bruker et Node.js Alpine-bilde som inkluderer de nødvendige verktøyene og miljøet for å bygge applikasjonen. Det andre stadiet bruker et distroless-bilde (FROM gcr.io/distroless/nodejs:18) som kun inneholder kjøretidsavhengighetene. Denne separasjonen sikrer at byggeavhengighetene ikke er til stede i det endelige bildet, noe som reduserer sikkerhetstrusler.