Skip to content

Commit f981196

Browse files
authored
fiks sentry source maps (#6905)
* fiks sentry source maps * fiks knip warning * fiks sentry dsn * test at feil logges * legg til knapp for å simulere en error * legg til sentry auth token som input til build * legg til VITE_SENTRY_AUTH_TOKEN i turbo config * fiks vite env variable forwarding i turbo.json * rydd opp i kode
1 parent 264bf45 commit f981196

File tree

15 files changed

+275
-97
lines changed

15 files changed

+275
-97
lines changed

.github/workflows/build.yml

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -88,13 +88,7 @@ jobs:
8888
run: yarn ci --filter=@navikt/${{ inputs.app }}
8989
env:
9090
VITE_SENTRY_RELEASE: ${{ steps.sentry_release_key.outputs.SENTRY_RELEASE }}
91-
92-
- name: Opprett release med Sentry
93-
if: inputs.push-image
94-
run: yarn sentry-release
95-
env:
96-
SENTRY_RELEASE: ${{ steps.sentry_release_key.outputs.SENTRY_RELEASE }}
97-
SENTRY_AUTH_TOKEN: ${{ secrets.SENTRY_AUTH_TOKEN }}
91+
VITE_SENTRY_AUTH_TOKEN: ${{ inputs.push-image && secrets.SENTRY_AUTH_TOKEN || '' }}
9892

9993
- name: Bygg server
10094
run: cd ./server && yarn install --immutable && yarn build

apps/fp-avdelingsleder/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -38,6 +38,7 @@
3838
"@navikt/ft-form-validators": "4.2.14",
3939
"@navikt/ft-ui-komponenter": "6.2.11",
4040
"@navikt/ft-utils": "3.8.13",
41+
"@sentry/vite-plugin": "4.6.0",
4142
"@tanstack/react-query": "5.90.6",
4243
"@tanstack/react-query-devtools": "5.90.2",
4344
"dayjs": "1.11.19",

apps/fp-avdelingsleder/src/main.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ if (app === null) {
1717
throw new Error('No app element');
1818
}
1919

20-
initSentry();
20+
initSentry('fp-avdelingsleder');
2121

2222
const root = createRoot(app);
2323

apps/fp-avdelingsleder/vite.config.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
/// <reference types="vitest" />
22
import { mergeConfig } from 'vite';
3+
import { sentryVitePlugin } from '@sentry/vite-plugin';
34

45
import { createSharedAppConfig } from '@navikt/fp-config-vite';
56

@@ -27,4 +28,17 @@ export default mergeConfig(createSharedAppConfig(), {
2728
},
2829
},
2930
},
31+
plugins: [
32+
// Put the Sentry vite plugin after all other plugins
33+
sentryVitePlugin({
34+
authToken: process.env.VITE_SENTRY_AUTH_TOKEN, // Kommer fra Github organization secrets
35+
disable: !process.env.VITE_SENTRY_AUTH_TOKEN, // Ikke last opp source maps hvis token ikke er satt. Token er bare satt når det bygges fra master branch
36+
org: 'nav',
37+
project: 'fp-avdelingsleder',
38+
url: 'https://sentry.gc.nav.no',
39+
release: {
40+
name: process.env.VITE_SENTRY_RELEASE,
41+
},
42+
}),
43+
],
3044
});

apps/fp-frontend/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -112,6 +112,7 @@
112112
"@navikt/ft-ui-komponenter": "6.2.11",
113113
"@navikt/ft-utils": "3.8.13",
114114
"@popperjs/core": "2.11.8",
115+
"@sentry/vite-plugin": "4.6.0",
115116
"@tanstack/react-query": "5.90.6",
116117
"@tanstack/react-query-devtools": "5.90.2",
117118
"classnames": "2.5.1",

apps/fp-frontend/src/main.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ if (app === null) {
1717
throw new Error('No app element');
1818
}
1919

20-
initSentry();
20+
initSentry('fp-frontend');
2121

2222
const root = createRoot(app);
2323

apps/fp-frontend/vite.config.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
/// <reference types="vitest" />
22
import { mergeConfig } from 'vite';
3+
import { sentryVitePlugin } from '@sentry/vite-plugin';
34

45
import { createSharedAppConfig } from '@navikt/fp-config-vite';
56

@@ -38,4 +39,17 @@ export default mergeConfig(createSharedAppConfig(), {
3839
},
3940
},
4041
},
42+
plugins: [
43+
// Put the Sentry vite plugin after all other plugins
44+
sentryVitePlugin({
45+
authToken: process.env.VITE_SENTRY_AUTH_TOKEN, // Kommer fra Github organization secrets
46+
disable: !process.env.VITE_SENTRY_AUTH_TOKEN, // Ikke last opp source maps hvis token ikke er satt. Token er bare satt når det bygges fra master branch
47+
org: 'nav',
48+
project: 'fp-frontend',
49+
url: 'https://sentry.gc.nav.no',
50+
release: {
51+
name: process.env.VITE_SENTRY_RELEASE,
52+
},
53+
}),
54+
],
4155
});

apps/fp-journalforing/package.json

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -36,6 +36,7 @@
3636
"@navikt/ft-form-validators": "4.2.14",
3737
"@navikt/ft-ui-komponenter": "6.2.11",
3838
"@navikt/ft-utils": "3.8.13",
39+
"@sentry/vite-plugin": "4.6.0",
3940
"@tanstack/react-query": "5.90.6",
4041
"@tanstack/react-query-devtools": "5.90.2",
4142
"dayjs": "1.11.19",

apps/fp-journalforing/src/main.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@ const app = document.getElementById('root');
1616
if (app === null) {
1717
throw new Error('No app element');
1818
}
19-
initSentry();
19+
initSentry('fp-journalforing');
2020

2121
const root = createRoot(app);
2222

apps/fp-journalforing/vite.config.ts

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
/// <reference types="vitest" />
22
import { mergeConfig } from 'vite';
3+
import { sentryVitePlugin } from '@sentry/vite-plugin';
34

45
import { createSharedAppConfig } from '@navikt/fp-config-vite';
56

@@ -27,4 +28,17 @@ export default mergeConfig(createSharedAppConfig(), {
2728
},
2829
},
2930
},
31+
plugins: [
32+
// Put the Sentry vite plugin after all other plugins
33+
sentryVitePlugin({
34+
authToken: process.env.VITE_SENTRY_AUTH_TOKEN, // Kommer fra Github organization secrets
35+
disable: !process.env.VITE_SENTRY_AUTH_TOKEN, // Ikke last opp source maps hvis token ikke er satt. Token er bare satt når det bygges fra master branch
36+
org: 'nav',
37+
project: 'fp-journalforing',
38+
url: 'https://sentry.gc.nav.no',
39+
release: {
40+
name: process.env.VITE_SENTRY_RELEASE,
41+
},
42+
}),
43+
],
3044
});

0 commit comments

Comments
 (0)