Only do string trim when it's necessary

This will enhance performance a little bit.

Only do string trim when it's necessary
fbcbb77f · Cao Shufeng · bcccbc3d · fbcbb77f
Commit fbcbb77f authored Jun 15, 2017 by Cao Shufeng
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 3 deletions

default.go pkg/auth/nodeidentifier/default.go +2 -3

No files found.
--- a/pkg/auth/nodeidentifier/default.go
+++ b/pkg/auth/nodeidentifier/default.go
@@ -50,8 +50,6 @@ func (defaultNodeIdentifier) NodeIdentity(u user.Info) (string, bool) {
 		return "", false
 	}

-	nodeName := strings.TrimPrefix(userName, nodeUserNamePrefix)
-
 	isNode := false
 	for _, g := range u.GetGroups() {
 		if g == user.NodesGroup {
@@ -63,5 +61,6 @@ func (defaultNodeIdentifier) NodeIdentity(u user.Info) (string, bool) {
 		return "", false
 	}

-	return nodeName, isNode
+	nodeName := strings.TrimPrefix(userName, nodeUserNamePrefix)
+	return nodeName, true
 }