From 72282a07df0b2d019795fc172aaa1070f7bfc896 Mon Sep 17 00:00:00 2001
From: Jakob Ackermann <jakob.ackermann@overleaf.com>
Date: Fri, 28 Aug 2020 11:47:45 +0100
Subject: [PATCH] [misc] socket.io: use a custom logger

- forward the previously enabled log messages to our logger-module
- stub the previously disabled logger methods
- drop the log-level config for socket.io
---
 services/real-time/app.js | 18 ++++++++++++++++--
 1 file changed, 16 insertions(+), 2 deletions(-)

diff --git a/services/real-time/app.js b/services/real-time/app.js
index 993bb58ff6..7525ed4d11 100644
--- a/services/real-time/app.js
+++ b/services/real-time/app.js
@@ -23,11 +23,26 @@ const CookieParser = require('cookie-parser')
 const DrainManager = require('./app/js/DrainManager')
 const HealthCheckManager = require('./app/js/HealthCheckManager')
 
+// NOTE: debug is invoked for every blob that is put on the wire
+const socketIoLogger = {
+  error(...message) {
+    logger.info({ fromSocketIo: true, originalLevel: 'error' }, ...message)
+  },
+  warn(...message) {
+    logger.info({ fromSocketIo: true, originalLevel: 'warn' }, ...message)
+  },
+  info() {},
+  debug() {},
+  log() {}
+}
+
 // Set up socket.io server
 const app = express()
 
 const server = require('http').createServer(app)
-const io = require('socket.io').listen(server)
+const io = require('socket.io').listen(server, {
+  logger: socketIoLogger
+})
 
 // Bind to sessions
 const sessionStore = new RedisStore({ client: sessionRedisClient })
@@ -61,7 +76,6 @@ io.configure(function () {
     'xhr-polling',
     'jsonp-polling'
   ])
-  io.set('log level', 1)
 })
 
 app.get('/', (req, res) => res.send('real-time-sharelatex is alive'))