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 <[email protected]>
Gerrit-Reviewer: coren <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
