Skip to content

Commit 202eebf

Browse files
committedOct 2, 2023
Add unit test and remove not needed parts
1 parent a70b5f6 commit 202eebf

File tree

2 files changed

+13
-21
lines changed

2 files changed

+13
-21
lines changed
 

‎doc/aaa/ldap/hld_ldap.md

+13-14
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,6 @@ LDAP_TABLE:{
117117
" hostname_check ": {{False}}
118118
" ldap_version": {{3}}
119119
" user_base_dn": {{ou=users,dc=example,dc=com (string)}}
120-
" login_name_attribute ": {{num}}
121120
" ldap_port": {{389}}
122121
" referrals": {{enabled}}
123122
" timeout": {{5 (duration_sec)}}
@@ -145,10 +144,6 @@ uid=6000(test_admin), groups=1000(admin),4(adm),27(sudo),999(docker),1001(redis)
145144

146145
not relevant
147146

148-
### Configuration and management
149-
TODO restapi?
150-
ngmi?
151-
152147
#### Manifest (if the feature is an Application Extension)
153148

154149
not relevant
@@ -286,13 +281,6 @@ module sonic-system-ldap {
286281
description "Ldap user base dn";
287282
}
288283
289-
leaf login_name_attribute {
290-
type string {
291-
length "1..65";
292-
}
293-
description "Ldap login name attribute";
294-
}
295-
296284
leaf ldap_port {
297285
type inet:port-number;
298286
default 389;
@@ -412,7 +400,7 @@ Example of error flows:
412400
• Timeout
413401
• Loss of network connectivity
414402

415-
When LDAP login failed as result of authentication timeout. If configured fallback enabled(TBD: check if default in SONiC). The login flow will do the next follow the authentication method detected.
403+
When LDAP login failed as result of authentication timeout the login flow will do the next following authentication method detected. If configured fallback enabled.
416404
Authentication failed – user will not be able to connect like regular authentication fail.
417405

418406
### Restrictions/Limitations
@@ -443,7 +431,18 @@ AAA LDAP login configuration:
443431
- Configure AAA authentication login local, ldap
444432

445433
#### Unit Test cases
446-
TODO
434+
#### Unit Test cases
435+
Test changes in database affect end-point configuration.
436+
Check config files after all config are set
437+
- Configure base-dn
438+
- Configure bind-dn
439+
- Configure bind-password
440+
- Configure ldap server-ip
441+
- Configure group-attribute
442+
- Configure ldap port
443+
- Configure scope
444+
- Configure timeout
445+
- Configure version
447446

448447
#### System Test cases
449448

‎doc/aaa/ldap/sonic-system-ldap.yang

-7
Original file line numberDiff line numberDiff line change
@@ -127,13 +127,6 @@ module sonic-system-ldap {
127127
description "Ldap user base dn";
128128
}
129129

130-
leaf login_name_attribute {
131-
type string {
132-
length "1..65";
133-
}
134-
description "Ldap login name attribute";
135-
}
136-
137130
leaf ldap_port {
138131
type inet:port-number;
139132
default 389;

0 commit comments

Comments
 (0)