return non-empty value if trimming is not required

Signed-off-by: pgbv <[email protected]>
sftpgo · Aug 21, 2024 · bbb7149 · bbb7149
1 parent 1513afd
commit bbb7149
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 0 deletions.
diff --git a/authenticator/ldap_test.go b/authenticator/ldap_test.go
@@ -352,6 +352,8 @@ func TestGetCNFromDN(t *testing.T) {
 	res = getCNFromDN("cn=test")
 	assert.Equal(t, "test", res)
 	res = getCNFromDN("test")
+	assert.Equal(t, "test", res)
+	res = getCNFromDN("")
 	assert.Empty(t, res)
 	res = getCNFromDN("cn=admin ,ou=users,dc=mylab,dc=local")
 	assert.Equal(t, "admin", res)

diff --git a/authenticator/utils.go b/authenticator/utils.go
@@ -85,6 +85,7 @@ func getCNFromDN(dn string) string {
 		if strings.HasPrefix(cn, "cn=") || strings.HasPrefix(cn, "ou=") {
 			return strings.TrimSpace(cn[3:])
 		}
+		return cn
 	}
 	return ""
 }