[Updater] Replace unmaintained winston-sentry-log with `winston-tra…

…nsport-sentry-node`
UniversityOfHelsinkiCS · Nov 8, 2024 · b4e176a · b4e176a
1 parent ce982e3
commit b4e176a
Show file tree

Hide file tree

Showing 8 changed files with 2,773 additions and 350 deletions.
diff --git a/updater/sis-updater-scheduler/package-lock.json b/updater/sis-updater-scheduler/package-lock.json
diff --git a/updater/sis-updater-scheduler/package.json b/updater/sis-updater-scheduler/package.json
@@ -15,9 +15,9 @@
     "node-nats-streaming": "^0.3.2",
     "pg": "^8.11.5",
     "redis": "^4.7.0",
-    "winston": "^3.13.0",
+    "winston": "^3.16.0",
     "winston-gelf-transporter": "^1.0.2",
-    "winston-sentry-log": "^1.0.26"
+    "winston-transport-sentry-node": "^3.0.0"
   },
   "devDependencies": {
     "nodemon": "^3.1.0"

diff --git a/updater/sis-updater-scheduler/src/config.js b/updater/sis-updater-scheduler/src/config.js
@@ -22,9 +22,4 @@ module.exports.ENABLE_WORKER_REPORTING =
   module.exports.EXIT_AFTER_IMMEDIATES || process.env.ENABLE_WORKER_REPORTING === 'yes'
 module.exports.SLACK_WEBHOOK = process.env.SLACK_WEBHOOK
 module.exports.serviceProvider = process.env.SERVICE_PROVIDER ? process.env.SERVICE_PROVIDER.toLowerCase() : ''
-
-// Sentry
-module.exports.sentryRelease = process.env.SENTRY_RELEASE || ''
-module.exports.sentryEnvironment = process.env.SENTRY_ENVIRONMENT || ''
-module.exports.sentryDSN = process.env.SENTRY_DSN || ''
 module.exports.runningInCI = process.env.CI === 'true'
diff --git a/updater/sis-updater-scheduler/src/utils/logger.js b/updater/sis-updater-scheduler/src/utils/logger.js
@@ -1,36 +1,18 @@
 const os = require('os')
 const winston = require('winston')
 const { WinstonGelfTransporter } = require('winston-gelf-transporter')
-const Sentry = require('winston-sentry-log')
+const Sentry = require('winston-transport-sentry-node').default
 
-const {
-  isDev,
-  isStaging,
-  isProduction,
-  sentryRelease,
-  sentryEnvironment,
-  sentryDSN,
-  runningInCI,
-  serviceProvider,
-} = require('../config')
+const { isDev, isStaging, isProduction, runningInCI, serviceProvider } = require('../config')
 
 const { combine, timestamp, printf, splat } = winston.format
 
 const formatDate = timestamp => new Date(timestamp).toLocaleString('fi-FI')
 
 const transports = []
 
-if (isProduction && !isStaging && !runningInCI && sentryDSN) {
-  const options = {
-    config: {
-      dsn: sentryDSN,
-      environment: sentryEnvironment,
-      release: sentryRelease,
-    },
-    level: 'error',
-  }
-
-  transports.push(new Sentry(options))
+if (isProduction && !isStaging && !runningInCI && process.env.SENTRY_DSN) {
+  transports.push(new Sentry({ level: 'error' }))
 }
 
 if (isDev) {