Skip to content

Commit

Permalink
GH-52 # Fix deprecated Sentry methods
Browse files Browse the repository at this point in the history
Bump Node to latest LTS (20)
  • Loading branch information
Hugo-C committed Mar 10, 2024
1 parent b695835 commit 6ce2e22
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 7 deletions.
4 changes: 0 additions & 4 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,6 @@ services:
container_name: jarm_online_gui_container
ports:
- "80:80"
depends_on:
- jarm_online_api

jarm_online_api:
build: .
Expand All @@ -22,8 +20,6 @@ services:
- ROCKET_SENTRY_DSN=${ROCKET_SENTRY_DSN:-https://[email protected]/123}
ports:
- "8000:8000"
depends_on:
- redis

redis:
image: redis:7-alpine
Expand Down
2 changes: 1 addition & 1 deletion jarm_online_gui/Dockerfile
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Taken from https://cli.vuejs.org/guide/deployment.html#docker-nginx
FROM node:18 as build-stage
FROM node:20 as build-stage
ARG VUE_APP_SENTRY_DSN
RUN echo "Sentry dsn set to $VUE_APP_SENTRY_DSN"
WORKDIR /app
Expand Down
4 changes: 2 additions & 2 deletions jarm_online_gui/src/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,8 @@ Sentry.init({
app,
dsn: sentry_dsn,
integrations: [
new Sentry.BrowserTracing(),
new Sentry.Replay(),
Sentry.browserTracingIntegration(),
Sentry.replayIntegration(),
],
tracesSampleRate: 1.0,
replaysSessionSampleRate: 0.1,
Expand Down

0 comments on commit 6ce2e22

Please sign in to comment.