trustchain-oc1-orchestral/authentik
Archived
10
0
Fork 0
Code Issues Pull requests Packages Projects Releases Wiki Activity

outposts/ldap: fix panic in type conversion when value is nil

Browse source 
Signed-off-by: Jens Langhammer <jens.langhammer@beryju.org>
This commit is contained in:
Jens Langhammer 2022-04-11 15:52:55 +02:00
parent 677621989a
commit 9b6e47e6b8
2 changed files with 10 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
internal/outpost/ldap/utils/utils.go
View file

@ -29,7 +29,9 @@ func ldapResolveTypeSingle(in interface{}) *string {
s := BoolToString(*t)
return &s
default:
log.WithField("type", reflect.TypeOf(in).String()).Warning("Type can't be mapped to LDAP yet")
if in != nil {
log.WithField("type", reflect.TypeOf(in).String()).Warning("Type can't be mapped to LDAP yet")
}
return nil
}
}

9
internal/outpost/ldap/utils/utils_test.go
View file

@ -7,6 +7,11 @@ import (
"goauthentik.io/api/v3"
)
func Test_ldapResolveTypeSingle_nil(t *testing.T) {
var ex *string
assert.Equal(t, ex, ldapResolveTypeSingle(nil))
}
func TestAKAttrsToLDAP_String(t *testing.T) {
var d *map[string]interface{}
@ -54,7 +59,7 @@ func TestAKAttrsToLDAP_Dict(t *testing.T) {
assert.Equal(t, 1, len(AKAttrsToLDAP(d)))
assert.Equal(t, "foo", AKAttrsToLDAP(d)[0].Name)
// Dicts are currently unsupported, but make sure we don't crash
// assert.Equal(t, []string{nil}, AKAttrsToLDAP(d)[0].Values)
assert.Equal(t, []string([]string(nil)), AKAttrsToLDAP(d)[0].Values)
}
func TestAKAttrsToLDAP_Mixed(t *testing.T) {
@ -68,5 +73,5 @@ func TestAKAttrsToLDAP_Mixed(t *testing.T) {
assert.Equal(t, 1, len(AKAttrsToLDAP(d)))
assert.Equal(t, "foo", AKAttrsToLDAP(d)[0].Name)
// Dicts are currently unsupported, but make sure we don't crash
// assert.Equal(t, []string{nil}, AKAttrsToLDAP(d)[0].Values)
assert.Equal(t, []string{"foo", ""}, AKAttrsToLDAP(d)[0].Values)
}