Legg til faro params som env variabler istedet for å bruke nais.js so…

…m blir injected med svart magi
navikt · Dec 11, 2024 · 7531120 · 7531120
1 parent eb4a51b
commit 7531120
Show file tree

Hide file tree

Showing 9 changed files with 21 additions and 14 deletions.
diff --git a/.env.example b/.env.example
@@ -21,5 +21,8 @@ SANITY_PROJECT_ID="rt6o382n"
 SANITY_ACCESS_TOKEN=""
 SANITY_DATASET="development"
 
+FARO_URL="http://localhost:12347/collect"
+GITHUB_SHA="local"
+
 UNLEASH_SERVER_API_TOKEN="" # Finn token her https://dagpenger-unleash-web.iap.nav.cloud.nais.io/admin/api
 UNLEASH_SERVER_API_URL=https://dagpenger-unleash-api.nav.cloud.nais.io/api
diff --git a/.nais/nais.yaml b/.nais/nais.yaml
@@ -90,6 +90,11 @@ spec:
     - name: UNLEASH_SERVER_API_URL
       value: {{unleash.url}}
 
+    - name: FARO_URL
+      value: {{faro_url}}
+    - name: GITHUB_SHA
+      value: {{githubSha}}
+
   envFrom:
     - secret: dp-saksbehandling-frontend-unleash-api-token
 

diff --git a/.nais/vars-dev.yaml b/.nais/vars-dev.yaml
@@ -22,6 +22,8 @@ sanity_project_id: "rt6o382n"
 sanity_access_token: ""
 sanity_dataset: "production"
 
+faro_url: "https://telemetry.ekstern.dev.nav.no/collect"
+
 unleash:
   url: https://dagpenger-unleash-api.nav.cloud.nais.io/api
   environment: development

diff --git a/.nais/vars-prod.yaml b/.nais/vars-prod.yaml
@@ -22,6 +22,8 @@ sanity_project_id: "rt6o382n"
 sanity_access_token: ""
 sanity_dataset: "production"
 
+faro_url: "https://telemetry.nav.no/collect"
+
 unleash:
   url: https://dagpenger-unleash-api.nav.cloud.nais.io/api
   environment: production

diff --git a/app/faro.ts b/app/faro.ts
@@ -3,8 +3,6 @@ import { TracingInstrumentation } from "@grafana/faro-web-tracing";
 
 import { getEnv } from "~/utils/env.utils";
 
-import nais from "./nais.js";
-
 let faro: Faro | null = null;
 
 export function initFaro() {
@@ -14,8 +12,11 @@ export function initFaro() {
 
   faro = initializeFaro({
     paused: getEnv("IS_LOCALHOST") === "true",
-    url: nais.telemetryCollectorURL,
-    app: nais.app,
+    url: getEnv("FARO_URL"),
+    app: {
+      name: "dp-saksbehandling-frontend",
+      version: getEnv("GITHUB_SHA"),
+    },
     sessionTracking: {
       enabled: true,
       persistent: true,

diff --git a/app/nais.js b/app/nais.js
diff --git a/app/root.tsx b/app/root.tsx
@@ -118,6 +118,8 @@ export async function loader({ request }: LoaderFunctionArgs) {
       INNTEKTREDIGERING_URL: process.env.INNTEKTREDIGERING_URL,
       GRISEN_URL: process.env.GRISEN_URL,
       DAGPENGER_NORGE_URL: process.env.DAGPENGER_NORGE_URL,
+      GITHUB_SHA: process.env.GITHUB_SHA,
+      FARO_URL: process.env.FARO_URL,
     },
   });
 }

diff --git a/app/utils/env.utils.ts b/app/utils/env.utils.ts
@@ -24,6 +24,8 @@ interface IEnv {
   NAIS_FRONTEND_TELEMETRY_COLLECTOR_URL: string;
   UNLEASH_SERVER_API_URL: string;
   UNLEASH_SERVER_API_TOKEN: string;
+  FARO_URL: string;
+  GITHUB_SHA: string;
 }
 
 export function getEnv(value: keyof IEnv) {

diff --git a/vite.config.ts b/vite.config.ts
@@ -23,9 +23,6 @@ export default defineConfig({
   build: {
     manifest: true,
     sourcemap: true,
-    rollupOptions: {
-      external: ["./nais.js"],
-    },
   },
   resolve: {
     alias: {