Bug 989 - WebUI support for new LDAPS feature

Review Request #366 — Created July 24, 2024 and submitted

pmurugaiyan
APV10
rel_apv_10_7
989
kdutta, mmiriam, pradeep, prajesh, shuinvy, tanya

Bug 989 - WebUI support for new LDAPS feature

The changes are tested locally. (DUT - https://192.168.85.144:8888)


Description From Last Updated

This has to be Distinguished Name (DN).Users might get confused with AD domain name. check with Ansuk on this.

prajeshprajesh

can we remove the space

kduttakdutta

I think Ansuk added plain LDAP as well. es3. Please check with him

tanyatanya

can we have random example than exposing actual groups in our AD.

prajeshprajesh

please remove spaces here and couple more places below

tanyatanya
prajesh
  1. 
      
  2. This has to be Distinguished Name (DN).Users might get confused with AD domain name. check with Ansuk on this.

    1. Sure Praveen, updated to Distinguished Name.

  3. 
      
pmurugaiyan
kdutta
kdutta
  1. Ship It!
    1. just remove unnecessary space and commit

  2. 
      
pmurugaiyan
tanya
  1. 
      
  2. I think Ansuk added plain LDAP as well. es3. Please check with him

    1. Sure Tanya, as discussed will take that as separate activity after this.

  3. please remove spaces here and couple more places below

  4. 
      
pmurugaiyan
shuinvy
  1. Ship It!
  2. 
      
pmurugaiyan
tanya
  1. Ship It!
  2. 
      
pmurugaiyan
Review request changed

Status: Closed (submitted)

prajesh
  1. 
      
  2. can we have random example than exposing actual groups in our AD.

    1. Sure Praveen, will update with LDAP changes in the next PR.

  3. 
      
prajesh
  1. Ship It!
  2. 
      
Loading...