Adding payload validation for global proxy

components/mobile-plugins/android-plugin/org.wso2.carbon.device.mgt.mobile.android.api/src/main/java/org/wso2/carbon/mdm/services/android/bean/GlobalProxy.java

@@ -21,6 +21,7 @@ package org.wso2.carbon.mdm.services.android.bean;
 
 import io.swagger.annotations.ApiModel;
 import io.swagger.annotations.ApiModelProperty;
+import org.apache.commons.lang.StringUtils;
 
 import java.io.Serializable;
 
@@ -66,6 +67,22 @@ public class GlobalProxy extends AndroidOperation implements Serializable {
     )
     private String proxyPacUrl;
 
+    public boolean validateRequest() {
+        if (ProxyType.MANUAL.equals(this.proxyConfigType)) {
+            if (StringUtils.isEmpty(this.proxyHost)) {
+                return false;
+            }
+            if (this.proxyPort < 0 || this.proxyPort > 65535) {
+                return false;
+            }
+        } else if (ProxyType.AUTO.equals(this.proxyConfigType)) {
+            if (StringUtils.isEmpty(proxyPacUrl)) {
+                return false;
+            }
+        }
+        return false;
+    }
+
     public ProxyType getProxyConfigType() {
         return proxyConfigType;
     }

components/mobile-plugins/android-plugin/org.wso2.carbon.device.mgt.mobile.android.api/src/main/java/org/wso2/carbon/mdm/services/android/services/impl/DeviceManagementAdminServiceImpl.java

@@ -995,14 +995,21 @@ public class DeviceManagementAdminServiceImpl implements DeviceManagementAdminSe
             }
 
             GlobalProxy globalProxy = globalProxyBeanWrapper.getOperation();
-            ProfileOperation operation = new ProfileOperation();
+            if (globalProxy.validateRequest()) {
-            operation.setCode(AndroidConstants.OperationCodes.GLOBAL_PROXY);
+                ProfileOperation operation = new ProfileOperation();
-            operation.setType(Operation.Type.PROFILE);
+                operation.setCode(AndroidConstants.OperationCodes.GLOBAL_PROXY);
-            operation.setPayLoad(globalProxy.toJSON());
+                operation.setType(Operation.Type.PROFILE);
-
+                operation.setPayLoad(globalProxy.toJSON());
-            Activity activity = AndroidDeviceUtils
+
-                    .getOperationResponse(globalProxyBeanWrapper.getDeviceIDs(), operation);
+                Activity activity = AndroidDeviceUtils
-            return Response.status(Response.Status.CREATED).entity(activity).build();
+                        .getOperationResponse(globalProxyBeanWrapper.getDeviceIDs(), operation);
+                return Response.status(Response.Status.CREATED).entity(activity).build();
+            } else {
+                String errorMessage = "The payload of the global proxy operation is incorrect";
+                log.error(errorMessage);
+                throw new BadRequestException(
+                        new ErrorResponse.ErrorResponseBuilder().setCode(400L).setMessage(errorMessage).build());
+            }
         } catch (InvalidDeviceException e) {
             String errorMessage = "Invalid Device Identifiers found.";
             log.error(errorMessage, e);