From eefc99e8073e3c73141404d651d15d5c968647df Mon Sep 17 00:00:00 2001 From: Ryan Malloy Date: Sat, 6 Jun 2026 10:14:33 -0600 Subject: [PATCH] Remove temporary debug logging --- hscontrol/oidc.go | 3 --- hscontrol/types/users.go | 4 ---- 2 files changed, 7 deletions(-) diff --git a/hscontrol/oidc.go b/hscontrol/oidc.go index cebff35c..db89539d 100644 --- a/hscontrol/oidc.go +++ b/hscontrol/oidc.go @@ -276,7 +276,6 @@ func (a *AuthProviderOIDC) OIDCCallbackHandler( httpUserError(writer, fmt.Errorf("decoding ID token claims: %w", err)) return } - log.Debug().Int("groups_count_from_id_token", len(claims.Groups)).Strs("groups", claims.Groups).Msg("DEBUG: claims.Groups after id_token decode") // Fetch user information (email, groups, name, etc) from the userinfo endpoint // https://openid.net/specs/openid-connect-core-1_0.html#UserInfo @@ -300,11 +299,9 @@ func (a *AuthProviderOIDC) OIDCCallbackHandler( claims.Name = cmp.Or(userinfo2.Name, claims.Name) claims.ProfilePictureURL = cmp.Or(userinfo2.Picture, claims.ProfilePictureURL) - log.Debug().Int("groups_count_userinfo", len(userinfo2.Groups)).Strs("groups", userinfo2.Groups).Msg("DEBUG: userinfo2.Groups after userinfo decode") if userinfo2.Groups != nil { claims.Groups = userinfo2.Groups } - log.Debug().Int("groups_count_after_merge", len(claims.Groups)).Strs("groups", claims.Groups).Msg("DEBUG: claims.Groups after userinfo merge") } else { util.LogErr(err, "could not get userinfo; only using claims from id token") } diff --git a/hscontrol/types/users.go b/hscontrol/types/users.go index dd3a2d33..b2455a67 100644 --- a/hscontrol/types/users.go +++ b/hscontrol/types/users.go @@ -488,9 +488,5 @@ func (u *User) FromClaim(claims *OIDCClaims, emailVerifiedRequired bool) { u.Provider = util.RegisterMethodOIDC // Store OIDC groups for role-based access control - log.Debug(). - Int("groups_count", len(claims.Groups)). - Strs("groups", claims.Groups). - Msg("FromClaim: SetGroups input") u.SetGroups(claims.Groups) }