Feature: add Sentry to the node server #164

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged

ysbrandB merged 3 commits into main from feature/sentry/server

Jan 26, 2026

client/package-lock.json

Large diffs are not rendered by default.

client/package.json

-Original file line number
+Diff line change
@@ Expand Up / @@ -14,7 +14,7 @@ @@
         "@vitejs/plugin-vue": "^6.0.2",
         "core-js": "^3.21.1",
         "gsap": "^3.9.1",
-        "jquery": "^3.6.0",
+        "jquery": "^4.0.0",
         "mitt": "^3.0.0",
         "roboto-fontface": "*",
         "sass": "^1.95.0",
@@ Expand Down @@

package-lock.json

Large diffs are not rendered by default.

server/.env.example

-Original file line number
+Diff line change
@@ Expand Up / @@ -7,6 +7,9 @@ HTTPS=true @@
     # Current runtime environment
     NODE_ENV=development
+    #Sentry
+    SENTRY_DSN=
     # Name prefix for docker containers
     DOCKER_CONTAINER_NAME=protube
@@ Expand Down @@

server/app.js

-Original file line number
+Diff line change
@@ Expand Up @@
       );
       process.exit(1);
     }
     require("dotenv").config();
+    require("./instrument");
     global.logger = require("./utils/logger");
     global.enums = require("./utils/Enums").enums;
     const { EventEmitter } = require("events");
@@ Expand Down @@

server/instrument.js

-Original file line number
+Diff line change
@@ -0,0 +1,6 @@
+    const Sentry = require("@sentry/node");
+    // Ensure to call this before requiring any other modules!
+    Sentry.init({
+      dsn: process.env.SENTRY_DSN,
+      sendDefaultPii: false,
+    });

server/modules/AuthService.js

-Original file line number
+Diff line change
@@ -1,5 +1,4 @@
     const passport = require("passport");
-    const fetch = require("node-fetch");
     const OAuth2Strategy = require("passport-oauth2").Strategy;
     const { User } = require("./DataBase");
@@ Expand Down Expand Up / @@ -27,7 +26,9 @@ passport.use( @@
           );
           if (!response.ok) {
-            logger.serverError(`User fetch failed:  ${response.status}, ${response.statusText}`);
+            logger.serverError(
+              `User fetch failed:  ${response.status}, ${response.statusText}`
+            );
             return done(null, false);
           }
@@ Expand Down @@

server/modules/PlaybackManager.js

-Original file line number
+Diff line change
@@ -1,6 +1,5 @@
     const queueManager = require("./QueueManager");
     const radio = require("./RadioStations");
-    const fetch = require("node-fetch");
     const { isEmpty } = require("lodash");
     const { format_hh_mm_ss } = require("../utils/time-formatter");
@@ Expand Down @@

server/modules/RadioStations.js

-Original file line number
+Diff line change
@@ -1,5 +1,3 @@
-    const fetch = require("node-fetch");
     let allRadioStations = [];
     exports.getAllRadioStations = () => allRadioStations;
@@ Expand Down @@

server/modules/socket_endpoints/ScreenSocket.js

-Original file line number
+Diff line change
@@ -1,4 +1,3 @@
-    const fetch = require("node-fetch");
     const endpoint = io.of("/socket/screen");
     const queueManager = require("../QueueManager");
     const playbackManager = require("../PlaybackManager");
@@ Expand Down @@

Provide feedback