You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@cloudstack.apache.org by GitBox <gi...@apache.org> on 2018/01/10 10:15:48 UTC

[GitHub] SudharmaJain opened a new pull request #2400: CLOUDSTACK-8900 listLdapUsers with listType=new only filters users for logged-in user account

SudharmaJain opened a new pull request #2400: CLOUDSTACK-8900 listLdapUsers with listType=new only filters users for logged-in user account
URL: https://github.com/apache/cloudstack/pull/2400
 
 
   Added an ldap user 'ldapuser1' to admin account. Now on clicking 'Add LDAP Account' button, UI displays the ldap users other than 'ldapuser1'.  
   
   ![image](https://user-images.githubusercontent.com/12229259/34766536-b615f47c-f61a-11e7-9a3c-8e7e0d7ee7ad.png)
   
   ![image](https://user-images.githubusercontent.com/12229259/34766552-c0c5e8e6-f61a-11e7-968d-29cac3a046f1.png)
   
   Added a new admin account as 'admin2'. Login as admin2 and clicking on  'Add LDAP Account' displays all the ldap users including 'ldapuser1'. 
   
   ## Expected behavior
   
   List Ldapusers with listtype=new should filter already added users irrespective of logged in user.
   
   ## Before the Fix
   
   ![image](https://user-images.githubusercontent.com/12229259/34766571-decf6a1a-f61a-11e7-85d9-2c7cf712600d.png)
   ## After the Fix
   ![image](https://user-images.githubusercontent.com/12229259/34766581-e3293aa0-f61a-11e7-9a51-4828a4fb575c.png)
   

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services