diff options
| author |   Pamela Dragosh <pdragosh@research.att.com> | 2018-04-12 13:54:22 +0000 |
|---|---|---|
| committer |   Gerrit Code Review <gerrit@onap.org> | 2018-04-12 13:54:22 +0000 |
| commit | 214338aa61978daf5c169f02c4c0e56527a3f90d (patch) | |
| tree | 76ebdea3f5edd627a50da35e10110edd7c6056ed | |
| parent | 1c6d62ab27dcb3491cf52f939b06232a5c64c1ba (diff) | |
| parent | 89e06a653ef40d5fc91ad89be4722e02d67d8ebd (diff) | |
Merge "Fix Fortify Header Manipulation Issue"
| -rw-r--r-- | ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/handler/DeleteHandler.java | 9 |
1 files changed, 8 insertions, 1 deletions
diff --git a/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/handler/DeleteHandler.java b/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/handler/DeleteHandler.java index 85b6e24a1..f3dda33fc 100644 --- a/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/handler/DeleteHandler.java +++ b/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/handler/DeleteHandler.java @@ -64,7 +64,7 @@ public class DeleteHandler { public static final String POLICY_IN_PDP = "PolicyInPDP"; public static final String ERROR = "error"; public static final String UNKNOWN = "unknown"; - + private static final String REGEX = "[0-9a-zA-Z._]*"; public void doAPIDeleteFromPAP(HttpServletRequest request, HttpServletResponse response) throws IOException, SQLException { // get the request content into a String @@ -320,6 +320,13 @@ public class DeleteHandler { String groupId = request.getParameter("groupId"); String responseString = null; + if(groupId != null && !groupId.matches(REGEX) ){ + response.setStatus(HttpServletResponse.SC_BAD_REQUEST); + response.addHeader("error",ERROR); + response.addHeader("message", "Group Id is not valid"); + return; + } + PolicyLogger.info("JSON request from API to Delete Policy from the PDP: " + policyName); // for PUT operations the group may or may not need to exist before the operation can be done |