On 09/27/2012 02:15 PM, [email protected] wrote:
From: Michal Fojtik <[email protected]>
Signed-off-by: Michal fojtik <[email protected]>
---
server/lib/deltacloud/drivers/openstack/openstack_driver.rb | 7 +++++++
1 file changed, 7 insertions(+)
diff --git a/server/lib/deltacloud/drivers/openstack/openstack_driver.rb
b/server/lib/deltacloud/drivers/openstack/openstack_driver.rb
index 7e11258..86d6983 100644
--- a/server/lib/deltacloud/drivers/openstack/openstack_driver.rb
+++ b/server/lib/deltacloud/drivers/openstack/openstack_driver.rb
@@ -344,6 +344,9 @@ private
#for v2 authentication credentials.name == "username+tenant_name"
def new_client(credentials, type = :compute)
tokens = credentials.user.split("+")
+ if credentials.user.empty?
+ raise AuthenticationFailure.new(Exception.new("Error: you must supply
the username"))
+ end
if (tokens.size != 2 && api_v2)
raise ValidationFailure.new(Exception.new("Error: expected
\"username+tenantname\" as username, you provided: #{credentials.user}"))
else
@@ -505,6 +508,10 @@ private
status 400
end
+ on /Must supply a :username/ do
+ status 401
+ end
+
on /OpenStack::Exception::Authentication/ do
status 401
end
ACK