From 63346d979c6e09d6af55fe2ea8c22fed0a361976 Mon Sep 17 00:00:00 2001 From: Cao Shufeng Date: Fri, 3 Mar 2017 08:41:03 -0500 Subject: [PATCH] remove extra function call We have read user info from context in previous line. No need to call this function again. --- pkg/registry/rbac/validation/rule.go | 1 - 1 file changed, 1 deletion(-) diff --git a/pkg/registry/rbac/validation/rule.go b/pkg/registry/rbac/validation/rule.go index 17373bc004e..131c1820bcd 100644 --- a/pkg/registry/rbac/validation/rule.go +++ b/pkg/registry/rbac/validation/rule.go @@ -60,7 +60,6 @@ func ConfirmNoEscalation(ctx genericapirequest.Context, ruleResolver Authorizati ownerRightsCover, missingRights := Covers(ownerRules, rules) if !ownerRightsCover { - user, _ := genericapirequest.UserFrom(ctx) return apierrors.NewUnauthorized(fmt.Sprintf("attempt to grant extra privileges: %v user=%v ownerrules=%v ruleResolutionErrors=%v", missingRights, user, ownerRules, ruleResolutionErrors)) } return nil