From db4d9f05622d918e197ec3c2e7a891067115925d Mon Sep 17 00:00:00 2001
From: Jessica Lawshe <5312836+lawshe@users.noreply.github.com>
Date: Wed, 8 Mar 2023 08:43:24 -0600
Subject: [PATCH] Merge pull request #11784 from overleaf/jel-saml-log

[web] Only log SAMLResponse on error

GitOrigin-RevId: 4bc33fadf49d148fd089bf45f87dfce6b0f268c3
---
 .../web/app/src/Features/SamlLog/SamlLogHandler.js     | 10 +++++++---
 1 file changed, 7 insertions(+), 3 deletions(-)

diff --git a/services/web/app/src/Features/SamlLog/SamlLogHandler.js b/services/web/app/src/Features/SamlLog/SamlLogHandler.js
index 9e2b3a734c..48e4dd96e7 100644
--- a/services/web/app/src/Features/SamlLog/SamlLogHandler.js
+++ b/services/web/app/src/Features/SamlLog/SamlLogHandler.js
@@ -41,10 +41,14 @@ function log(req, data, samlAssertion) {
       samlLog.samlAssertion = JSON.stringify(samlAssertionForLog)
     }
 
-    if (data.error || samlAssertion) {
+    if (data.error) {
       data.body = {}
-      if (req.body.email) data.body.email = req.body.email
-      if (req.body.SAMLResponse) data.body.SAMLResponse = req.body.SAMLResponse
+      if (req.body.email) {
+        data.body.email = req.body.email
+      }
+      if (req.body.SAMLResponse) {
+        data.body.SAMLResponse = req.body.SAMLResponse
+      }
     }
 
     try {