Parse and dedup but do not lowercase principals in SSH certs. (#2591)

hashicorp · Apr 18, 2017 · 866b384 · 866b384
1 parent a385d1e
commit 866b384
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/builtin/logical/ssh/path_sign.go b/builtin/logical/ssh/path_sign.go
@@ -203,8 +203,8 @@ func (b *backend) calculateValidPrincipals(data *framework.FieldData, defaultPri
 		validPrincipals = defaultPrincipal
 	}
 
-	parsedPrincipals := strutil.ParseDedupAndSortStrings(validPrincipals, ",")
-	allowedPrincipals := strutil.ParseDedupAndSortStrings(principalsAllowedByRole, ",")
+	parsedPrincipals := strutil.RemoveDuplicates(strutil.ParseStringSlice(validPrincipals, ","), false)
+	allowedPrincipals := strutil.RemoveDuplicates(strutil.ParseStringSlice(principalsAllowedByRole, ","), false)
 	switch {
 	case len(parsedPrincipals) == 0:
 		// There is nothing to process