coren has submitted this change and it was merged. Change subject: Search for projects using ldap_list, part two. ......................................................................
Search for projects using ldap_list, part two. This is a continuation of change Ida578039f7395f88c2339d209bb14aa235696987 ... Last time I fixed this for the 'can see all projects' code path but not for the more commonly used normal user path. For bug 47101. Change-Id: I778fb78c1d877db1e3279aeadaaf2edde96ce32c --- M nova/OpenStackNovaUser.php 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: coren: Verified; Looks good to me, approved diff --git a/nova/OpenStackNovaUser.php b/nova/OpenStackNovaUser.php index deda742..15776d4 100644 --- a/nova/OpenStackNovaUser.php +++ b/nova/OpenStackNovaUser.php @@ -159,7 +159,7 @@ # roles do not $projects = array(); $filter = "(&(objectclass=groupofnames)(member=$this->userDN))"; - $result = LdapAuthenticationPlugin::ldap_search( $wgAuth->ldapconn, $wgOpenStackManagerLDAPProjectBaseDN, $filter ); + $result = LdapAuthenticationPlugin::ldap_list( $wgAuth->ldapconn, $wgOpenStackManagerLDAPProjectBaseDN, $filter ); if ( $result ) { $entries = LdapAuthenticationPlugin::ldap_get_entries( $wgAuth->ldapconn, $result ); if ( $entries ) { -- To view, visit https://gerrit.wikimedia.org/r/61783 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I778fb78c1d877db1e3279aeadaaf2edde96ce32c Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/extensions/OpenStackManager Gerrit-Branch: master Gerrit-Owner: Andrew Bogott <abog...@wikimedia.org> Gerrit-Reviewer: coren <mpellet...@wikimedia.org> Gerrit-Reviewer: jenkins-bot _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits