diff --git a/api/src/org/apache/cloudstack/api/response/FirewallResponse.java b/api/src/org/apache/cloudstack/api/response/FirewallResponse.java index 26d243386fc..13a04b68f0b 100644 --- a/api/src/org/apache/cloudstack/api/response/FirewallResponse.java +++ b/api/src/org/apache/cloudstack/api/response/FirewallResponse.java @@ -39,10 +39,10 @@ public class FirewallResponse extends BaseResponse { private String endPort; @SerializedName(ApiConstants.IP_ADDRESS_ID) @Param(description="the public ip address id for the firewall rule") - private Long publicIpAddressId; + private String publicIpAddressId; @SerializedName(ApiConstants.NETWORK_ID) @Param(description="the network id of the firewall rule") - private Long networkId; + private String networkId; @SerializedName(ApiConstants.IP_ADDRESS) @Param(description="the public ip address for the firewall rule") private String publicIpAddress; @@ -78,7 +78,7 @@ public class FirewallResponse extends BaseResponse { this.endPort = endPort; } - public void setPublicIpAddressId(Long publicIpAddressId) { + public void setPublicIpAddressId(String publicIpAddressId) { this.publicIpAddressId = publicIpAddressId; } @@ -86,7 +86,7 @@ public class FirewallResponse extends BaseResponse { this.publicIpAddress = publicIpAddress; } - public void setNetworkId(Long networkId) { + public void setNetworkId(String networkId) { this.networkId = networkId; } diff --git a/server/src/com/cloud/api/ApiResponseHelper.java b/server/src/com/cloud/api/ApiResponseHelper.java index 4f75b99e0c4..94c5d6c1fa1 100755 --- a/server/src/com/cloud/api/ApiResponseHelper.java +++ b/server/src/com/cloud/api/ApiResponseHelper.java @@ -2399,11 +2399,12 @@ public class ApiResponseHelper implements ResponseGenerator { if (fwRule.getTrafficType() == FirewallRule.TrafficType.Ingress) { IpAddress ip = ApiDBUtils.findIpAddressById(fwRule.getSourceIpAddressId()); - response.setPublicIpAddressId(ip.getId()); + response.setPublicIpAddressId(ip.getUuid()); response.setPublicIpAddress(ip.getAddress().addr()); } else if (fwRule.getTrafficType() == FirewallRule.TrafficType.Egress) { response.setPublicIpAddress(null); - response.setNetworkId(fwRule.getNetworkId()); + Network network = ApiDBUtils.findNetworkById(fwRule.getNetworkId()); + response.setNetworkId(network.getUuid()); } FirewallRule.State state = fwRule.getState();