apache/cloudstack
1
0
Fork 0
mirror of https://github.com/apache/cloudstack.git synced 2025-12-16 10:32:34 +01:00
Code Issues Packages Projects Releases Wiki Activity

Fixed authorizeSecurityGroupIngressRule to work with "name" parameter

Browse Source
This commit is contained in:
alena 2011-05-25 18:54:11 -07:00
parent cd6d933e48
commit 8a68e8148e
2 changed files with 10 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
api/src/com/cloud/api/commands/AuthorizeSecurityGroupIngressCmd.java
View File

@ -120,6 +120,11 @@ public class AuthorizeSecurityGroupIngressCmd extends BaseAsyncCmd {
if (securityGroupId == null) {
throw new InvalidParameterValueException("Unable to find security group " + securityGroupName + " for account id=" + getEntityOwnerId());
}
securityGroupName = null;
}
if (securityGroupId == null) {
throw new InvalidParameterValueException("Either securityGroupId or securityGroupName is required by authorizeSecurityGroupIngress command");
}
return securityGroupId;

6
api/src/com/cloud/api/commands/DeleteSecurityGroupCmd.java
View File

@ -59,6 +59,10 @@ public class DeleteSecurityGroupCmd extends BaseCmd {
}
}
if (id == null) {
throw new InvalidParameterValueException("Either id or name parameter is requred by deleteSecurityGroup command");
}
return id;
}
@ -105,4 +109,4 @@ public class DeleteSecurityGroupCmd extends BaseCmd {
throw new ServerApiException(BaseCmd.RESOURCE_IN_USE_ERROR, ex.getMessage());
}
}
}
}