From: Pamela Dragosh <pdragosh@research.att.com>
Date: Thu, 1 Mar 2018 23:00:06 +0000 (-0500)
Subject: Remove CLM issues with commons-collections
X-Git-Tag: v1.2.0~101^2
X-Git-Url: https://gerrit.onap.org/r/gitweb?a=commitdiff_plain;h=refs%2Fchanges%2F97%2F33697%2F1;p=policy%2Fengine.git

Remove CLM issues with commons-collections

We know that we are not configuring an LDAP PIP in our
use of the XACML open source. The LDAP implementation
uses Apache Velocity, which uses a very old version
of commons-collections that has security issues. So
we can exclude commons-collections from the build.

Issue-ID: POLICY-507
Change-Id: I735eae4fe507ad016d9b0b49e67536415edb9820
Signed-off-by: Pamela Dragosh <pdragosh@research.att.com>
---

diff --git a/ONAP-PDP/pom.xml b/ONAP-PDP/pom.xml
index d302c955d..dc3953b76 100644
--- a/ONAP-PDP/pom.xml
+++ b/ONAP-PDP/pom.xml
@@ -73,6 +73,15 @@
 			<groupId>com.att.research.xacml</groupId>
 			<artifactId>xacml-pdp</artifactId>
 			<version>1.0.1</version>
+            <exclusions>
+            <!-- The LDAP PIP uses velocity which pulls this insecure jar in. We
+            are not using that PIP and can safely exclude this jar to resolve CLM issue.
+             -->
+              <exclusion>
+                <groupId>commons-collections</groupId>
+                <artifactId>commons-collections</artifactId>
+              </exclusion>
+            </exclusions>
 		</dependency>
 		<dependency>
 			<groupId>junit</groupId>
diff --git a/ONAP-XACML/pom.xml b/ONAP-XACML/pom.xml
index c399e3fa9..b6f12c005 100644
--- a/ONAP-XACML/pom.xml
+++ b/ONAP-XACML/pom.xml
@@ -83,6 +83,15 @@
 			<groupId>com.att.research.xacml</groupId>
 			<artifactId>xacml</artifactId>
 			<version>1.0.1</version>
+            <exclusions>
+            <!-- The LDAP PIP uses velocity which pulls this insecure jar in. We
+            are not using that PIP and can safely exclude this jar to resolve CLM issue.
+             -->
+              <exclusion>
+                <groupId>commons-collections</groupId>
+                <artifactId>commons-collections</artifactId>
+              </exclusion>
+            </exclusions>
 		</dependency>
 	</dependencies>
 </project>