alexvanin/certificates
1
0
Fork 0
forked from TrueCloudLab/certificates
Code Pull requests Activity

Simplify check of principals in a case insensitive way

Browse source 
Fixes #679
This commit is contained in:
Mariano Cano 2021-09-08 16:00:33 -07:00
parent 7a94b0c157
commit 141c519171
1 changed files with 2 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
authority/provisioner/sign_ssh_options.go
View file

@ -454,20 +454,12 @@ func containsAllMembers(group, subgroup []string) bool {
if lsg > lg || (lg > 0 && lsg == 0) { if lsg > lg || (lg > 0 && lsg == 0) {
return false return false
} }
groupLower := []string{}
subgroupLower := []string{}
for _, s := range group {
groupLower = append(groupLower, strings.ToLower(s))
}
for _, s := range subgroup {
subgroupLower = append(subgroupLower, strings.ToLower(s))
}
visit := make(map[string]struct{}, lg) visit := make(map[string]struct{}, lg)
for i := 0; i < lg; i++ { for i := 0; i < lg; i++ {
visit[groupLower[i]] = struct{}{} visit[strings.ToLower(group[i])] = struct{}{}
} }
for i := 0; i < lsg; i++ { for i := 0; i < lsg; i++ {
if _, ok := visit[subgroupLower[i]]; !ok { if _, ok := visit[strings.ToLower(subgroup[i])]; !ok {
return false return false
} }
} }