diff options
| author | James Elliott <james-d-elliott@users.noreply.github.com> | 2021-04-12 11:10:50 +1000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-04-12 11:10:50 +1000 |
| commit | 9e7b73bd565a29fbc13e525ec4487d39fe94226b (patch) | |
| tree | 51f80d08df87c56a15dbb42bdbdab60f1264ad1d /internal/authentication/ldap_user_provider_test.go | |
| parent | 1a385947bcd13270d5bc370dd03621e1d51671f8 (diff) | |
refactor(authentication): add trace logs for the user/group baseDN (#1904)
This logs the baseDN for user and group searching on startup as well as the users filter (with just input remaining). Additionally refactors the location of a few log messages, and exposes the logger to the provider to reduce calls to logging.Logger().
Diffstat (limited to 'internal/authentication/ldap_user_provider_test.go')
| -rw-r--r-- | internal/authentication/ldap_user_provider_test.go | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/internal/authentication/ldap_user_provider_test.go b/internal/authentication/ldap_user_provider_test.go index 4036415f7..05a6a60ea 100644 --- a/internal/authentication/ldap_user_provider_test.go +++ b/internal/authentication/ldap_user_provider_test.go @@ -725,8 +725,8 @@ func TestShouldParseDynamicConfiguration(t *testing.T) { assert.Equal(t, "(&(|(uid={input})(mail={input})(displayname={input}))(objectCategory=person)(objectClass=user)(!userAccountControl:1.2.840.113556.1.4.803:=2)(!pwdLastSet=0))", ldapClient.configuration.UsersFilter) assert.Equal(t, "(&(|(member={dn})(member={input})(member={username}))(objectClass=group))", ldapClient.configuration.GroupsFilter) - assert.Equal(t, "ou=users,dc=example,dc=com", ldapClient.usersDN) - assert.Equal(t, "ou=groups,dc=example,dc=com", ldapClient.groupsDN) + assert.Equal(t, "ou=users,dc=example,dc=com", ldapClient.usersBaseDN) + assert.Equal(t, "ou=groups,dc=example,dc=com", ldapClient.groupsBaseDN) } func TestShouldCallStartTLSWithInsecureSkipVerifyWhenSkipVerifyTrue(t *testing.T) { |