Yuvipanda has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/166032

Change subject: [WIP] Do not require auth for listing instance info
......................................................................

[WIP] Do not require auth for listing instance info

Change-Id: Idc43ece235cae461997608a70bbb866ad3ff7d61
---
M api/ApiListNovaInstances.php
1 file changed, 3 insertions(+), 1 deletion(-)


  git pull 
ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/OpenStackManager 
refs/changes/32/166032/1

diff --git a/api/ApiListNovaInstances.php b/api/ApiListNovaInstances.php
index be90e63..e414972 100644
--- a/api/ApiListNovaInstances.php
+++ b/api/ApiListNovaInstances.php
@@ -15,6 +15,8 @@
        }
 
        public function run() {
+               global $wgOpenStackManagerLDAPUsername;
+
                $params = $this->extractRequestParams();
                $project = OpenStackNovaProject::getProjectByName( 
$params['project'] );
                if ( !$project ) {
@@ -26,7 +28,7 @@
                        $this->dieUsage( 'Must be logged in to use this API', 
'notloggedin' );
                }
 
-               $user = new OpenStackNovaUser();
+               $user = new OpenStackNovaUser( $wgOpenStackManagerLDAPUsername 
);
                if ( !$user->exists() ) {
                        $this->dieUsage( 'NovaUser does not exist', 'baduser' );
                }

-- 
To view, visit https://gerrit.wikimedia.org/r/166032
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Idc43ece235cae461997608a70bbb866ad3ff7d61
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/OpenStackManager
Gerrit-Branch: master
Gerrit-Owner: Yuvipanda <yuvipa...@gmail.com>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to