From 8d569815e6b493bb2811a46e896fd541cea93dc0 Mon Sep 17 00:00:00 2001
From: Jakob Ackermann <jakob.ackermann@overleaf.com>
Date: Thu, 13 Mar 2025 07:51:51 +0000
Subject: [PATCH] [real-time] only log userId from session (#24281)

While we are at it, log projectId and transport.

GitOrigin-RevId: ac10220247947e0050f1374c81091c50c8cc0e85
---
 services/real-time/app/js/Router.js | 26 ++++++++++++++------------
 1 file changed, 14 insertions(+), 12 deletions(-)

diff --git a/services/real-time/app/js/Router.js b/services/real-time/app/js/Router.js
index e99369955f..e907a1b684 100644
--- a/services/real-time/app/js/Router.js
+++ b/services/real-time/app/js/Router.js
@@ -216,18 +216,6 @@ module.exports = Router = {
       })
       metrics.gauge('socket-io.clients', io.sockets.clients().length)
 
-      const info = {
-        session,
-        publicId: client.publicId,
-        clientId: client.id,
-        isDebugging,
-      }
-      if (isDebugging) {
-        logger.info(info, 'client connected')
-      } else {
-        logger.debug(info, 'client connected')
-      }
-
       let user
       if (session && session.passport && session.passport.user) {
         ;({ user } = session.passport)
@@ -238,6 +226,20 @@ module.exports = Router = {
         user = { _id: 'anonymous-user', anonymousAccessToken }
       }
 
+      const info = {
+        userId: user._id,
+        projectId,
+        transport: client.transport,
+        publicId: client.publicId,
+        clientId: client.id,
+        isDebugging,
+      }
+      if (isDebugging) {
+        logger.info(info, 'client connected')
+      } else {
+        logger.debug(info, 'client connected')
+      }
+
       const connectionDetails = {
         userId: user._id,
         projectId,