commit b05c409d0982debee393de8087a4c5a9e9d5373e
author Edwin Kempin <ekempin@google.com> Mon Jul 22 11:52:33 2024 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> Mon Jul 22 11:52:33 2024 +0000
tree f5e88d5db29110b1bd9ecf4e4407c405d87e8d05
parent 223434b9df3516600d370f4bc09becdd7929891f
parent ded8d88b15d8924504787e102c11cb71ae298837

Merge "IdentifiedUser: Do not log known groups when tracing"

java/com/google/gerrit/server/IdentifiedUser.java

diff --git a/java/com/google/gerrit/server/IdentifiedUser.java b/java/com/google/gerrit/server/IdentifiedUser.java
index d45d329..b069e39 100644
--- a/java/com/google/gerrit/server/IdentifiedUser.java
+++ b/java/com/google/gerrit/server/IdentifiedUser.java
@@ -16,7 +16,6 @@
 
 import static com.google.common.base.MoreObjects.firstNonNull;
 import static com.google.common.collect.ImmutableSet.toImmutableSet;
-import static com.google.common.flogger.LazyArgs.lazy;
 
 import com.google.common.annotations.VisibleForTesting;
 import com.google.common.collect.ImmutableSet;
@@ -422,13 +421,9 @@
     if (effectiveGroups == null) {
       if (authConfig.isIdentityTrustable(state().externalIds())) {
         effectiveGroups = groupBackend.membershipsOf(this);
-        logger.atFinest().log(
-            "Known groups of %s: %s", getLoggableName(), lazy(effectiveGroups::getKnownGroups));
       } else {
         effectiveGroups = registeredGroups;
-        logger.atFinest().log(
-            "%s has a non-trusted identity, falling back to %s as known groups",
-            getLoggableName(), lazy(registeredGroups::getKnownGroups));
+        logger.atFinest().log("%s has a non-trusted identity", getLoggableName());
       }
     }
     return effectiveGroups;