change claims logging to trace

DuendeSoftware · Sep 23, 2022 · f009311 · f009311
1 parent 992af62
commit f009311
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/Duende.Bff/EndpointServices/BackchannelLogout/DefaultBackchannelLogoutService.cs b/src/Duende.Bff/EndpointServices/BackchannelLogout/DefaultBackchannelLogoutService.cs
@@ -127,8 +127,7 @@ await UserSession.RevokeSessionsAsync(new UserSessionsFilter
         }
         else
         {
-            // TODO: any sensitive data here, or are we ok with LogDebug?
-            Logger.LogDebug("Claims found in back-channel JWT {claims}", claims.Claims);
+            Logger.LogTrace("Claims found in back-channel JWT {claims}", claims.Claims);
         }
 
         if (claims.FindFirst("sub") == null && claims.FindFirst("sid") == null)

diff --git a/src/Duende.Bff/EndpointServices/User/DefaultUserService.cs b/src/Duende.Bff/EndpointServices/User/DefaultUserService.cs
@@ -80,8 +80,7 @@ public virtual async Task ProcessRequestAsync(HttpContext context)
             context.Response.ContentType = "application/json";
             await context.Response.WriteAsync(json, Encoding.UTF8);
 
-            // TODO: any PII filtering we need here, or are we ok since it's Debug?
-            Logger.LogDebug("User endpoint indicates the user is logged in with claims {claims}", claims);
+            Logger.LogTrace("User endpoint indicates the user is logged in with claims {claims}", claims);
         }
     }