Bruk service discovery for å nå forebyggingsplan

.env.development

@@ -2,7 +2,6 @@ DECORATOR_BREADCRUMB_THIS_PAGE_URL=http://localhost:3000/forebygge-fravar
 DECORATOR_ENV=dev
 FIA_ARBEIDSGIVER_HOSTNAME=arbeidsgiver.ekstern.dev.nav.no
 SYKEFRAVARSSTATISTIKK_URL=/forebygge-fravar/sykefravarsstatistikk
-FOREBYGGINGSPLAN_URL=https://arbeidsgiver.ekstern.dev.nav.no/forebyggingsplan
 MIN_SIDE_ARBEIDSGIVER_URL=https://arbeidsgiver.ekstern.dev.nav.no/min-side-arbeidsgiver
 KONTAKT_OSS_ARBEIDSGIVER_URL=https://arbeidsgiver-kontakt-oss.dev.nav.no/kontakt-oss
 GRAFANA_AGENT_COLLECTOR_URL=http://localhost:12347/collect

.env.test

@@ -2,7 +2,6 @@ DECORATOR_BREADCRUMB_THIS_PAGE_URL=https://forebygge-fravar.intern.dev.nav.no/fo
 DECORATOR_ENV=dev
 FIA_ARBEIDSGIVER_HOSTNAME=fia-arbeidsgiver.test.url
 SYKEFRAVARSSTATISTIKK_URL=https://sykefravarsstatistikk.test.url
-FOREBYGGINGSPLAN_URL=https://forebyggingsplan.test.url
 MIN_SIDE_ARBEIDSGIVER_URL=https://min-side-arbeidsgiver.test.url
 KONTAKT_OSS_ARBEIDSGIVER_URL=https://kontakt-oss.test.url
 GRAFANA_AGENT_COLLECTOR_URL=https://telemetry.ekstern.dev.nav.no/collect

.github/workflows/build-deploy.yaml

@@ -4,7 +4,7 @@ on:
   push:
     branches:
       - main
-      - router-upgrade
+      - bruk-service-discovery-til-forebyggingsplan
     paths-ignore:
       - "**.md"
       - "**/**.md"

.nais/dev.yaml

@@ -14,7 +14,7 @@ spec:
     - name: SYKEFRAVARSSTATISTIKK_API_HOSTNAME
       value: sykefravarsstatistikk-api.dev-fss-pub.nais.io
     - name: FOREBYGGINGSPLAN_API_BASEURL
-      value: forebyggingsplan.teamia.svc.cluster.local
+      value: forebyggingsplan.teamia
     - name: FOREBYGGINGSPLAN_API_AUDIENCE
       value: dev-gcp:teamia:forebyggingsplan
     - name: IA_TJENESTER_METRIKKER_HOSTNAME
@@ -47,8 +47,6 @@ spec:
       value: fia-arbeidsgiver.pia
     - name: SYKEFRAVARSSTATISTIKK_URL
       value: https://forebygge-fravar.intern.dev.nav.no/forebygge-fravar/sykefravarsstatistikk
-    - name: FOREBYGGINGSPLAN_URL
-      value: https://forebyggingsplan-frontend.intern.dev.nav.no/forebyggingsplan
     - name: MIN_SIDE_ARBEIDSGIVER_URL
       value: https://arbeidsgiver.intern.dev.nav.no/min-side-arbeidsgiver
     - name: KONTAKT_OSS_ARBEIDSGIVER_URL

.nais/ekstern-dev.yaml

@@ -27,8 +27,6 @@ spec:
       value: https://arbeidsgiver.ekstern.dev.nav.no/forebygge-fravar
     - name: SYKEFRAVARSSTATISTIKK_URL
       value: https://arbeidsgiver.ekstern.dev.nav.no/forebygge-fravar/sykefravarsstatistikk
-    - name: FOREBYGGINGSPLAN_URL
-      value: https://arbeidsgiver.ekstern.dev.nav.no/forebyggingsplan
     - name: MIN_SIDE_ARBEIDSGIVER_URL
       value: https://arbeidsgiver.ekstern.dev.nav.no/min-side-arbeidsgiver
     - name: KONTAKT_OSS_ARBEIDSGIVER_URL

.nais/prod.yaml

@@ -43,8 +43,6 @@ spec:
       value: fia-arbeidsgiver.pia
     - name: SYKEFRAVARSSTATISTIKK_URL
       value: https://arbeidsgiver.nav.no/forebygge-fravar/sykefravarsstatistikk
-    - name: FOREBYGGINGSPLAN_URL
-      value: https://arbeidsgiver.nav.no/forebyggingsplan
     - name: MIN_SIDE_ARBEIDSGIVER_URL
       value: https://arbeidsgiver.nav.no/min-side-arbeidsgiver
     - name: KONTAKT_OSS_ARBEIDSGIVER_URL

src/Aktiviteter/status-klient.tsx

@@ -1,12 +1,13 @@
 import { StatusType } from "./AktivitetData";
+import {API_BASE_PATH} from "../utils/konstanter";
 
 export const oppdaterStatus = async (
   aktivitetId: string,
   orgnr: string,
   status: StatusType
 ) => {
   return await fetch(
-    `forebygge-fravar/api/aktivitet/${aktivitetId}/orgnr/${orgnr}/oppdater`,
+      `${API_BASE_PATH}/aktivitet/${aktivitetId}/orgnr/${orgnr}/oppdater`,
     {
       method: "POST",
       body: JSON.stringify({ status }),

src/hooks/useHentAktiviteter.ts

@@ -1,4 +1,5 @@
 import { useRestRessursSWR } from "./useRestRessursSWR";
+import {API_BASE_PATH} from "../utils/konstanter";
 
 export type StatusType = "AVBRUTT" | "STARTET" | "FULLFØRT";
 type AktivitetType = "OPPGAVE" | "AKTIVITETSKORT";
@@ -9,8 +10,10 @@ export type AktivitetBrukerStatus = {
 };
 
 export const useHentAktiviteter = (orgnr: string | null | undefined) => {
+  const apiPath = `${API_BASE_PATH}/aktiviteter/orgnr/${orgnr}`;
+
   return useRestRessursSWR<AktivitetBrukerStatus[]>(
-    `forebygge-fravar/api/aktiviteter/orgnr/${orgnr}`,
+      apiPath,
     "Det oppstod en feil ved henting av aktiviteter"
   );
 };

src/pages/api/aktivitet/[aktivitetId]/orgnr/[orgnr]/oppdater.ts

@@ -1,6 +1,6 @@
 import { NextApiRequest, NextApiResponse } from "next";
 import { erGyldigOrgnr } from "../../../../../../hooks/useOrgnr";
-import { backendLogger } from "../../../../../../utils/backendLogger";
+import {anonymizeOrgnr, backendLogger} from "../../../../../../utils/backendLogger";
 import {
   exchangeIdportenSubjectToken,
   isInvalidToken,
@@ -44,8 +44,10 @@ export default async function handler(
     return res.status(401).json({ error: "authentication failed" });
   }
 
+  const oppdaterUrl = `http://${process.env.FOREBYGGINGSPLAN_API_BASEURL}/aktivitet/${aktivitetId}/orgnr/${orgnr}/oppdater`;
+  backendLogger.info(`Kaller oppdatering av aktivitet på URL '${anonymizeOrgnr(oppdaterUrl)}'`);
   const respons = await fetch(
-    `http://${process.env.FOREBYGGINGSPLAN_API_BASEURL}/aktivitet/${aktivitetId}/orgnr/${orgnr}/oppdater`,
+      oppdaterUrl,
     {
       method: "POST",
       headers: {
@@ -56,5 +58,6 @@ export default async function handler(
     },
   );
 
+  backendLogger.info(`Fikk følgende status på /oppdater: '${respons.status}'`);
   return res.status(respons.status).json({ status: respons.status });
 }

src/pages/api/aktiviteter/orgnr/[orgnr].ts

@@ -1,6 +1,7 @@
 import { NextApiRequest, NextApiResponse } from "next";
 import { erGyldigOrgnr } from "../../../../hooks/useOrgnr";
 import proxyRequestWithTokenExchange from "../../../../utils/api-proxy";
+import {anonymizeOrgnr, backendLogger} from "../../../../utils/backendLogger";
 
 export default async function handler(
   req: NextApiRequest,
@@ -12,11 +13,14 @@ export default async function handler(
     return res.status(400).json({ error: "Mangler gyldig 'orgnr' i path" });
   }
 
+  const hostnavn = `${process.env.FOREBYGGINGSPLAN_API_BASEURL}`;
+  const sti = `/aktiviteter/orgnr/${orgnr}`;
+  backendLogger.info(`Kaller oppdatering av aktivitet på hostname '${hostnavn}' og sti '${anonymizeOrgnr(sti)}'`);
   return await proxyRequestWithTokenExchange(
     req,
     res,
-    `${process.env.FOREBYGGINGSPLAN_API_BASEURL}`,
-    `/aktiviteter/orgnr/${orgnr}`,
+      hostnavn,
+      sti,
     process.env.FOREBYGGINGSPLAN_API_AUDIENCE,
     false
   );

src/utils/backendLogger.test.tsx

@@ -0,0 +1,20 @@
+import { anonymizeOrgnr } from "./backendLogger";
+
+describe("Tester anonymize orgnr", () => {
+    it("Erstatter et orgnr med 9 stjerner", async () => {
+        expect(anonymizeOrgnr("Dette er et orgnr: 987654321"))
+            .toBe("Dette er et orgnr: *********");
+    });
+    it("Kan erstatteflere orgnr", async () => {
+        expect(anonymizeOrgnr("Dette er et orgnr: 987654321, og et annet orgnr: 123456789"))
+            .toBe("Dette er et orgnr: *********, og et annet orgnr: *********");
+    });
+    it("Erstatter ikke enkelte tall", async () => {
+        expect(anonymizeOrgnr("Dette er et kortere tall: 12345678"))
+            .toBe("Dette er et kortere tall: 12345678");
+    });
+    it("Erstatter ikke enkelte tall blandet med orgn", async () => {
+        expect(anonymizeOrgnr("Dette er et orgnr: 987654321, men ikke det her: 12345678"))
+            .toBe("Dette er et orgnr: *********, men ikke det her: 12345678");
+    });
+});

src/utils/backendLogger.ts

@@ -24,3 +24,8 @@ export const backendLogger = pino({
     },
   },
 });
+
+export const anonymizeOrgnr = (message: string) => {
+  return message
+      .replace(/\d{9}/g, "*********")
+}

src/utils/envUtils.ts

@@ -26,7 +26,6 @@ export const getProdUrl = (path?: string) => {
 
 export type Tjeneste =
   | "Sykefraværsstatistikk"
-  | "Forebyggingsplan"
   | "Min Side Arbeidsgiver"
   | "Kontakt Oss";
 
@@ -42,9 +41,6 @@ export const hentUrlFraMiljøvariabel = (tjeneste: Tjeneste) => {
     case "Kontakt Oss":
       url = process.env.KONTAKT_OSS_ARBEIDSGIVER_URL;
       break;
-    case "Forebyggingsplan":
-      url = process.env.FOREBYGGINGSPLAN_URL;
-      break;
   }
   if (!url) {
     throw Error(