From 58077c4eff455193dff4bce9e14030e5f61bc3da Mon Sep 17 00:00:00 2001
From: Ersun Warncke <ersun.warncke@overleaf.com>
Date: Wed, 18 Dec 2019 12:42:02 -0400
Subject: [PATCH] truncate sessionId in saml logs

GitOrigin-RevId: 45c369a88323e340875ae7b9f4d26e664ccdc8d8
---
 .../app/src/Features/SamlLog/SamlLogHandler.js   | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/services/web/app/src/Features/SamlLog/SamlLogHandler.js b/services/web/app/src/Features/SamlLog/SamlLogHandler.js
index 6f338d1990..91e70c6674 100644
--- a/services/web/app/src/Features/SamlLog/SamlLogHandler.js
+++ b/services/web/app/src/Features/SamlLog/SamlLogHandler.js
@@ -2,19 +2,19 @@ const { SamlLog } = require('../../models/SamlLog')
 const logger = require('logger-sharelatex')
 
 function log(providerId, sessionId, data) {
-  const samlLog = new SamlLog()
-  samlLog.providerId = (providerId || '').toString()
-  samlLog.sessionId = sessionId
   try {
+    const samlLog = new SamlLog()
+    samlLog.providerId = (providerId || '').toString()
+    samlLog.sessionId = (sessionId || '').toString().substr(0, 8)
     samlLog.jsonData = JSON.stringify(data)
+    samlLog.save(err => {
+      if (err) {
+        logger.error({ err, sessionId, providerId }, 'SamlLog Error')
+      }
+    })
   } catch (err) {
     logger.error({ err, sessionId, providerId }, 'SamlLog JSON.stringify Error')
   }
-  samlLog.save(err => {
-    if (err) {
-      logger.error({ err, sessionId, providerId }, 'SamlLog Error')
-    }
-  })
 }
 
 const SamlLogHandler = {