TrueCloudLab/certificates
17
0
Fork 2
Code Issues 1 Pull requests 1 Releases Activity Actions

Converts group and subgroup to lowercase for comparison.

Browse source 
Fixes #679
This commit is contained in:
Fearghal O Floinn 2021-09-08 12:24:49 +01:00
parent efac0926df
commit 7a94b0c157
1 changed files with 11 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

13
authority/provisioner/sign_ssh_options.go
View file

@ -5,6 +5,7 @@ import (
"encoding/binary" "encoding/binary"
"encoding/json" "encoding/json"
"math/big" "math/big"
"strings"
"time" "time"
"github.com/pkg/errors" "github.com/pkg/errors"
@ -453,12 +454,20 @@ 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[group[i]] = struct{}{} visit[groupLower[i]] = struct{}{}
} }
for i := 0; i < lsg; i++ { for i := 0; i < lsg; i++ {
if _, ok := visit[subgroup[i]]; !ok { if _, ok := visit[subgroupLower[i]]; !ok {
return false return false
} }
} }