X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=ONAP-PAP-REST%2Fsrc%2Fmain%2Fjava%2Forg%2Fonap%2Fpolicy%2Fpap%2Fxacml%2Frest%2FXACMLPapServlet.java;h=1acec1bd91b4764c2a6ca2f366e3f613113341d9;hb=8cf5ed300190ab386d95758e6d4be6c38e67123d;hp=84cd72fe6911dbebca10731b6221279a83d1b75b;hpb=9f19ef6e4c962160eb844da5f505be2fa3901a69;p=policy%2Fengine.git

diff --git a/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/XACMLPapServlet.java b/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/XACMLPapServlet.java
index 84cd72fe6..1acec1bd9 100644
--- a/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/XACMLPapServlet.java
+++ b/ONAP-PAP-REST/src/main/java/org/onap/policy/pap/xacml/rest/XACMLPapServlet.java
@@ -792,7 +792,6 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 		} else {
 			PolicyLogger.info("requestID was provided in call to XACMLPapSrvlet (doGet)");
 		}
-		try {
 			loggingContext.metricStarted();
 			XACMLRest.dumpRequest(request);
 			loggingContext.metricEnded();
@@ -907,7 +906,18 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 						request.getRemoteHost().equals(request.getLocalAddr())) {
 					// Return status information - basically all the groups
 					loggingContext.setServiceName("PAP.getGroups");
-					Set<OnapPDPGroup> groups = papEngine.getOnapPDPGroups();
+					Set<OnapPDPGroup> groups = null;
+					try {
+					    groups = papEngine.getOnapPDPGroups();
+					} catch(PAPException e) {
+					    LOGGER.debug(e);
+					    PolicyLogger.error(MessageCodes.ERROR_UNKNOWN, e, "XACMLPapServlet", " GET exception");
+					    loggingContext.transactionEnded();
+					    PolicyLogger.audit("Transaction Failed - See Error.log");
+					    setResponseError(response,HttpServletResponse.SC_INTERNAL_SERVER_ERROR, e.getMessage());
+					    im.endTransaction();
+					    return;
+					}
 					// convert response object to JSON and include in the response
 					mapperWriteValue(new ObjectMapper(), response,  groups);
 					response.setHeader("content-type", "application/json");
@@ -984,15 +994,6 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 				PolicyLogger.audit("Transaction Failed - See Error.log");
 				setResponseError(response,HttpServletResponse.SC_NOT_FOUND, message);
 			}
-		}  catch (PAPException e) {
-			LOGGER.debug(e);
-			PolicyLogger.error(MessageCodes.ERROR_UNKNOWN, e, "XACMLPapServlet", " GET exception");
-			loggingContext.transactionEnded();
-			PolicyLogger.audit("Transaction Failed - See Error.log");
-			setResponseError(response,HttpServletResponse.SC_INTERNAL_SERVER_ERROR, e.getMessage());
-			im.endTransaction();
-			return;
-		}
 		loggingContext.transactionEnded();
 		PolicyLogger.audit("Transaction Ended");
 		im.endTransaction();
@@ -1894,7 +1895,16 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 				} else {
 					// request is for top-level properties about all groups
 					loggingContext.setServiceName("AC:PAP.getAllGroups");
-					Set<OnapPDPGroup> groups = papEngine.getOnapPDPGroups();
+					Set<OnapPDPGroup> groups = null;
+					try {
+					    groups = papEngine.getOnapPDPGroups();
+					} catch(PAPException e) {
+					    PolicyLogger.error(MessageCodes.ERROR_PROCESS_FLOW, e, "XACMLPapServlet", " AC Get exception");
+					    loggingContext.transactionEnded();
+					    PolicyLogger.audit("Transaction Failed - See Error.log");
+					    setResponseError(response,HttpServletResponse.SC_INTERNAL_SERVER_ERROR, e.getMessage());
+					    return;
+					}
 					// convert response object to JSON and include in the response
 					mapperWriteValue(new ObjectMapper(), response,  groups);
 					if (LOGGER.isDebugEnabled()) {
@@ -2041,7 +2051,12 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 				LOGGER.info("JSON request from AC: " + json);
 				// convert Object sent as JSON into local object
 				ObjectMapper mapper = new ObjectMapper();
-				Object objectFromJSON = mapper.readValue(json, StdPDP.class);
+				Object objectFromJSON = null;
+				try {
+				    objectFromJSON = mapper.readValue(json, StdPDP.class);
+				} catch(Exception e) {
+				    LOGGER.error(e);
+				}
 				if (pdpId == null ||
 						objectFromJSON == null ||
 						! (objectFromJSON instanceof StdPDP) ||
@@ -2146,7 +2161,12 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 				LOGGER.info("JSON request from AC: " + json);
 				// convert Object sent as JSON into local object
 				ObjectMapper mapper = new ObjectMapper();
-				Object objectFromJSON  = mapper.readValue(json, StdPDPGroup.class);
+				Object objectFromJSON = null;
+				try {
+				    objectFromJSON  = mapper.readValue(json, StdPDPGroup.class);
+				} catch(Exception e) {
+				    LOGGER.error(e);
+				}
 				if (objectFromJSON == null || ! (objectFromJSON instanceof StdPDPGroup) ||
 						! ((StdPDPGroup)objectFromJSON).getId().equals(group.getId())) {
 					PolicyLogger.error(MessageCodes.ERROR_DATA_ISSUE + " Group update had bad input. id=" + group.getId() + " objectFromJSON="+objectFromJSON);
@@ -3014,4 +3034,4 @@ public class XACMLPapServlet extends HttpServlet implements StdItemSetChangeList
 	public static void setMsPolicyName(String msPolicyName) {
 		XACMLPapServlet.msPolicyName = msPolicyName;
 	}
-}
+}
\ No newline at end of file