sureshanaparti commented on code in PR #9977:
URL: https://github.com/apache/cloudstack/pull/9977#discussion_r1860598164


##########
plugins/vpn/wireguard/src/main/java/org/apache/cloudstack/api/response/CreateWgVpnResponse.java:
##########
@@ -0,0 +1,139 @@
+package org.apache.cloudstack.api.response;
+
+import com.cloud.serializer.Param;
+import com.google.gson.annotations.SerializedName;
+import org.apache.cloudstack.acl.RoleType;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.BaseResponse;
+
+@SuppressWarnings("unused")
+public class CreateWgVpnResponse extends BaseResponse {
+
+    @SerializedName(ApiConstants.ID)
+    @Param(description = "the ID of the new wireguard vpn tunnel")
+    private String id;
+
+    @SerializedName(ApiConstants.PUBLIC_IP_ID)
+    @Param(description = "the public ip address of the vpn server")
+    private String publicIpId;
+
+    @SerializedName(ApiConstants.PUBLIC_IP)
+    @Param(description = "the public ip address of the vpn server")
+    private String publicIp;
+
+    @SerializedName("networkid")
+    @Param(description = "id of the network accessible by this VPN")
+    private String NetworkId;
+
+    @SerializedName("publicport")
+    @Param(description = "the public port used by this vpn server")
+    private Integer PublicPort;
+
+    @SerializedName("accountid")
+    @Param(description = "the id of the account of the vpn")
+    private String AccountId;
+
+    @SerializedName(ApiConstants.ACCOUNT)
+    @Param(description = "the account of the wireguard vpn")
+    private String accountName;
+
+    @SerializedName(ApiConstants.DOMAIN_ID)
+    @Param(description = "the domain id of the account of the wireguard vpn")
+    private String domainId;
+
+    @SerializedName(ApiConstants.DOMAIN)
+    @Param(description = "the domain name of the account of the wireguard vpn")
+    private String domainName;
+
+    @SerializedName(ApiConstants.FOR_DISPLAY)
+    @Param(description = "is vpn for display to the regular user", since = 
"4.4", authorized = {RoleType.Admin})
+    private Boolean forDisplay;
+
+    @SerializedName("ip4enable")
+    @Param(description = "does the vpn support ipv4 addresses")
+    private Boolean IP4Enable;

Review Comment:
   ```suggestion
       private Boolean ip4Enable;
   ```



##########
plugins/vpn/wireguard/src/main/java/org/apache/cloudstack/api/response/CreateWgVpnResponse.java:
##########
@@ -0,0 +1,139 @@
+package org.apache.cloudstack.api.response;
+
+import com.cloud.serializer.Param;
+import com.google.gson.annotations.SerializedName;
+import org.apache.cloudstack.acl.RoleType;
+import org.apache.cloudstack.api.ApiConstants;
+import org.apache.cloudstack.api.BaseResponse;
+
+@SuppressWarnings("unused")
+public class CreateWgVpnResponse extends BaseResponse {
+
+    @SerializedName(ApiConstants.ID)
+    @Param(description = "the ID of the new wireguard vpn tunnel")
+    private String id;
+
+    @SerializedName(ApiConstants.PUBLIC_IP_ID)
+    @Param(description = "the public ip address of the vpn server")
+    private String publicIpId;
+
+    @SerializedName(ApiConstants.PUBLIC_IP)
+    @Param(description = "the public ip address of the vpn server")
+    private String publicIp;
+
+    @SerializedName("networkid")
+    @Param(description = "id of the network accessible by this VPN")
+    private String NetworkId;
+
+    @SerializedName("publicport")
+    @Param(description = "the public port used by this vpn server")
+    private Integer PublicPort;
+
+    @SerializedName("accountid")
+    @Param(description = "the id of the account of the vpn")
+    private String AccountId;
+
+    @SerializedName(ApiConstants.ACCOUNT)
+    @Param(description = "the account of the wireguard vpn")
+    private String accountName;
+
+    @SerializedName(ApiConstants.DOMAIN_ID)
+    @Param(description = "the domain id of the account of the wireguard vpn")
+    private String domainId;
+
+    @SerializedName(ApiConstants.DOMAIN)
+    @Param(description = "the domain name of the account of the wireguard vpn")
+    private String domainName;
+
+    @SerializedName(ApiConstants.FOR_DISPLAY)
+    @Param(description = "is vpn for display to the regular user", since = 
"4.4", authorized = {RoleType.Admin})
+    private Boolean forDisplay;
+
+    @SerializedName("ip4enable")
+    @Param(description = "does the vpn support ipv4 addresses")
+    private Boolean IP4Enable;
+
+    @SerializedName("ip4range")
+    @Param(description = "IPv4 network to use internally for the vpn (CIDR 
notation)")
+    private String IP4Range;

Review Comment:
   ```suggestion
       private String ip4Range;
   ```



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to