aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authordc443y <dc443y@att.com>2022-07-06 14:25:55 -0500
committerdc443y <dc443y@att.com>2022-07-07 08:42:59 -0500
commit36dc881e3511787d1211bc60c433f4c29ac89648 (patch)
tree84bcbfd5daf5896e86c29ba1700cb1bd28b5bfb4
parentf9c88a07e28ab0495b621c36c8e4b71653f6c09f (diff)
Fix Raw XACML Decision API Serialization
Issue-ID: POLICY-4274 Signed-off-by: dc443y <dc443y@att.com> Change-Id: I0d367d35e7ec52c7587ec75194b654cc41303868
-rw-r--r--main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlJsonMessageBodyHandler.java4
-rw-r--r--main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlXmlMessageBodyHandler.java4
-rw-r--r--main/src/test/java/org/onap/policy/pdpx/main/rest/TestDecision.java8
3 files changed, 8 insertions, 8 deletions
diff --git a/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlJsonMessageBodyHandler.java b/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlJsonMessageBodyHandler.java
index 9b59a003..907e0f1a 100644
--- a/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlJsonMessageBodyHandler.java
+++ b/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlJsonMessageBodyHandler.java
@@ -1,6 +1,6 @@
/*-
* ============LICENSE_START=======================================================
- * Copyright (C) 2020 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2020-2022 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -98,6 +98,6 @@ public class XacmlJsonMessageBodyHandler implements MessageBodyReader<Request>,
return false;
}
return ("xacml+json".equals(mediaType.getSubtype()))
- && (type == Request.class || type == Response.class);
+ && (Request.class.isAssignableFrom(type) || Response.class.isAssignableFrom(type));
}
} \ No newline at end of file
diff --git a/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlXmlMessageBodyHandler.java b/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlXmlMessageBodyHandler.java
index ddd77528..7c1af4b4 100644
--- a/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlXmlMessageBodyHandler.java
+++ b/main/src/main/java/org/onap/policy/pdpx/main/rest/serialization/XacmlXmlMessageBodyHandler.java
@@ -1,6 +1,6 @@
/*-
* ============LICENSE_START=======================================================
- * Copyright (C) 2020 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2020-2022 AT&T Intellectual Property. All rights reserved.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -98,6 +98,6 @@ public class XacmlXmlMessageBodyHandler implements MessageBodyReader<Request>, M
}
return ("xacml+xml".equals(mediaType.getSubtype()))
- && (type == Request.class || type == Response.class);
+ && (Request.class.isAssignableFrom(type) || Response.class.isAssignableFrom(type));
}
} \ No newline at end of file
diff --git a/main/src/test/java/org/onap/policy/pdpx/main/rest/TestDecision.java b/main/src/test/java/org/onap/policy/pdpx/main/rest/TestDecision.java
index 2d71543c..f7f3e8a5 100644
--- a/main/src/test/java/org/onap/policy/pdpx/main/rest/TestDecision.java
+++ b/main/src/test/java/org/onap/policy/pdpx/main/rest/TestDecision.java
@@ -1,6 +1,6 @@
/*-
* ============LICENSE_START=======================================================
- * Copyright (C) 2019-2021 AT&T Intellectual Property. All rights reserved.
+ * Copyright (C) 2019-2022 AT&T Intellectual Property. All rights reserved.
* Modifications Copyright (C) 2019 Nordix Foundation.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
@@ -197,11 +197,11 @@ public class TestDecision {
String responseFromXmlRequest = getNativeDecision(xmlRequestAsString, APPLICATION_XACML_XML);
LOGGER.info("Response from xml request {}", responseFromXmlRequest);
- assertThat(responseFromXmlRequest).contains("NOTAPPLICABLE");
+ assertThat(responseFromXmlRequest).contains("<Response xmlns=").contains("NotApplicable");
String responseFromJsonRequest = getNativeDecision(jsonRequestAsString, APPLICATION_XACML_JSON);
LOGGER.info("Response from json request {}", responseFromJsonRequest);
- assertThat(responseFromJsonRequest).contains("NOTAPPLICABLE");
+ assertThat(responseFromJsonRequest).contains("\"Response\":").contains("NotApplicable");
}
private static Main startXacmlPdpService(File params) throws PolicyXacmlPdpException {
@@ -224,7 +224,7 @@ public class TestDecision {
private String getNativeDecision(String request, String mediaType) {
Entity<String> entityRequest = Entity.entity(request, mediaType);
- Response response = client.post("/xacml", entityRequest, Collections.emptyMap());
+ Response response = client.post("/xacml", entityRequest, Map.of("Accept", mediaType));
assertEquals(200, response.getStatus());