Fix warning message for invalid groups

louketo · Dec 10, 2018 · 91f6fdc · 91f6fdc
1 parent 0c7fe8b
commit 91f6fdc
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/middleware.go b/middleware.go
@@ -316,7 +316,7 @@ func (r *oauthProxy) admissionMiddleware(resource *Resource) func(http.Handler)
 
 			// @step: check if we have any groups, the groups are there
 			if !hasAccess(resource.Groups, user.groups, false) {
-				r.log.Warn("access denied, invalid roles",
+				r.log.Warn("access denied, invalid groups",
 					zap.String("access", "denied"),
 					zap.String("email", user.email),
 					zap.String("resource", resource.URL),

diff --git a/session.go b/session.go
@@ -52,7 +52,8 @@ func (r *oauthProxy) getIdentity(req *http.Request) (*userContext, error) {
 		zap.String("id", user.id),
 		zap.String("name", user.name),
 		zap.String("email", user.email),
-		zap.String("roles", strings.Join(user.roles, ",")))
+		zap.String("roles", strings.Join(user.roles, ",")),
+		zap.String("groups", strings.Join(user.groups, ",")))
 
 	return user, nil
 }