From 139f9b088e4bc02b4bc4ff6b37a1261ac0f4cadf Mon Sep 17 00:00:00 2001 From: Fiete Ostkamp Date: Wed, 21 Feb 2024 15:15:39 +0100 Subject: Update aai-common version to 1.13.4 in graphadmin - update to 1.13.4 - bump activemq dep to fix security vulnerability - update mockito from 1.X to 3.4 - bump graphadmin version to 1.13.4 to be in sync with aai-common Issue-ID: AAI-3784 Change-Id: Idcaf9b18ce6cb100b5db88ce0fc6d62157b6c387 Signed-off-by: Fiete Ostkamp --- pom.xml | 63 +++++++++++++--------- .../pre/RequestTransactionLogging.java | 7 ++- .../java/org/onap/aai/rest/util/EchoResponse.java | 4 +- .../org/onap/aai/MetricsConfigurationTest.java | 16 +++--- .../org/onap/aai/audit/AuditGraphson2SqlTest.java | 8 +-- .../datasnapshot/DataSnapshot4HistInitTest.java | 5 +- .../onap/aai/datasnapshot/DataSnapshotTest.java | 10 +--- .../datasnapshot/DataSnapshotTest4HistInit.java | 9 +--- .../aai/dbgen/schemamod/SchemaMod4HistTest.java | 17 ------ .../onap/aai/dbgen/schemamod/SchemaModTest.java | 17 ------ .../migration/v20/MigrateL2DefaultToFalseTest.java | 2 +- version.properties | 4 +- 12 files changed, 67 insertions(+), 95 deletions(-) diff --git a/pom.xml b/pom.xml index b15ee5a..59fcddd 100755 --- a/pom.xml +++ b/pom.xml @@ -26,16 +26,15 @@ org.onap.aai.aai-common aai-parent - 1.9.4 + 1.13.4 org.onap.aai.graphadmin aai-graphadmin - 1.12.3-SNAPSHOT + 1.13.4-SNAPSHOT - 1.10.19 1.8 1.8 1.8 @@ -56,7 +55,7 @@ localhost:5000 1.0.0 1.12.3 - 1.9.4 + 1.13.4 ${project.build.directory}/${project.artifactId}-${project.version}-build/ onap @@ -72,7 +71,7 @@ Nexus Proxy Properties and Snapshot Locations Ideally this can be overwritten at runtime per internal environment specific values at runtime --> - https://nexus.onap.org + https://nexus.onap.org /content/sites/site/org/onap/aai/graphadmin/${project.artifactId}/${project.version} /content/repositories/releases/ /content/repositories/snapshots/ @@ -107,7 +106,8 @@ 1.8.1 1.8.1 1.8.1 - 2.1.12.RELEASE + 5.16.7 + 4.9.3 @@ -342,6 +342,18 @@ + + + + + com.google.guava + guava + 19.0 + + + @@ -467,10 +479,6 @@ org.apache.tinkerpop tinkergraph-gremlin - - org.antlr - antlr4-runtime - org.apache.tinkerpop gremlin-core @@ -494,12 +502,6 @@ org.json json - - - javax.xml.bind - jaxb-api - - org.eclipse.persistence eclipselink @@ -521,10 +523,12 @@ org.apache.activemq activemq-broker + ${activemq.version} org.apache.activemq activemq-client + ${activemq.version} com.sun.jersey @@ -590,10 +594,6 @@ org.springframework.boot spring-boot-starter-jetty - - org.springframework.boot - spring-boot-starter-test - org.springframework.boot spring-boot-starter-jersey @@ -607,7 +607,6 @@ org.springframework.boot spring-boot-starter-actuator - ${spring.boot.starter.actuator.version} org.springframework.boot @@ -668,7 +667,7 @@ org.mockito - mockito-all + mockito-core test @@ -676,7 +675,23 @@ jsonassert test - + + org.springframework.boot + spring-boot-starter-test + test + + + + org.junit.vintage + junit-vintage-engine + test + + + org.hamcrest + hamcrest-core + + + @@ -971,7 +986,7 @@ org.antlr antlr4-maven-plugin - 4.7 + ${antlr.version} diff --git a/src/main/java/org/onap/aai/interceptors/pre/RequestTransactionLogging.java b/src/main/java/org/onap/aai/interceptors/pre/RequestTransactionLogging.java index 6aa180b..ecaff91 100644 --- a/src/main/java/org/onap/aai/interceptors/pre/RequestTransactionLogging.java +++ b/src/main/java/org/onap/aai/interceptors/pre/RequestTransactionLogging.java @@ -34,6 +34,7 @@ import org.springframework.beans.factory.annotation.Autowired; import javax.annotation.Priority; import javax.servlet.http.HttpServletRequest; +import javax.ws.rs.HttpMethod; import javax.ws.rs.container.ContainerRequestContext; import javax.ws.rs.container.ContainerRequestFilter; import javax.ws.rs.container.PreMatching; @@ -86,11 +87,13 @@ public class RequestTransactionLogging extends AAIContainerFilter implements Con String contentType = headersMap.getFirst(CONTENT_TYPE); String acceptType = headersMap.getFirst(ACCEPT); - if(contentType == null || contentType.contains(TEXT_PLAIN)){ + if(contentType == null && !requestContext.getMethod().equals(HttpMethod.GET.toString())){ + LOGGER.debug("Content Type header missing in the request, adding one of [{}]", DEFAULT_CONTENT_TYPE); requestContext.getHeaders().putSingle(CONTENT_TYPE, DEFAULT_CONTENT_TYPE); } - if(acceptType == null || acceptType.contains(TEXT_PLAIN)){ + if(acceptType == null){ + LOGGER.debug("Accept header missing in the request, adding one of [{}]", DEFAULT_RESPONSE_TYPE); requestContext.getHeaders().putSingle(ACCEPT, DEFAULT_RESPONSE_TYPE); } } diff --git a/src/main/java/org/onap/aai/rest/util/EchoResponse.java b/src/main/java/org/onap/aai/rest/util/EchoResponse.java index 05ff38e..91841df 100644 --- a/src/main/java/org/onap/aai/rest/util/EchoResponse.java +++ b/src/main/java/org/onap/aai/rest/util/EchoResponse.java @@ -94,7 +94,7 @@ public class EchoResponse extends RESTAPI { response = Response.status(Status.OK) .entity(ErrorLogHelper.getRESTAPIInfoResponse( - headers.getAcceptableMediaTypes(), exceptionList)) + new ArrayList<>(headers.getAcceptableMediaTypes()), exceptionList)) .build(); } catch (Exception e) { @@ -106,7 +106,7 @@ public class EchoResponse extends RESTAPI { response = Response .status(Status.INTERNAL_SERVER_ERROR) .entity(ErrorLogHelper.getRESTAPIErrorResponse( - headers.getAcceptableMediaTypes(), ex, + new ArrayList<>(headers.getAcceptableMediaTypes()), ex, templateVars)).build(); } finally { diff --git a/src/test/java/org/onap/aai/MetricsConfigurationTest.java b/src/test/java/org/onap/aai/MetricsConfigurationTest.java index d0c0ce4..68b2ca8 100644 --- a/src/test/java/org/onap/aai/MetricsConfigurationTest.java +++ b/src/test/java/org/onap/aai/MetricsConfigurationTest.java @@ -25,6 +25,7 @@ import static org.junit.Assert.assertTrue; import java.io.UnsupportedEncodingException; +import java.util.Arrays; import java.util.Base64; import java.util.Collections; import org.junit.Before; @@ -37,6 +38,7 @@ import org.onap.aai.exceptions.AAIException; import org.onap.aai.restclient.PropertyPasswordConfiguration; import org.springframework.beans.factory.annotation.Autowired; import org.springframework.boot.actuate.autoconfigure.web.server.LocalManagementPort; +import org.springframework.boot.test.autoconfigure.actuate.metrics.AutoConfigureMetrics; import org.springframework.boot.test.context.SpringBootTest; import org.springframework.boot.test.context.SpringBootTest.WebEnvironment; import org.springframework.boot.web.server.LocalServerPort; @@ -51,6 +53,7 @@ import org.springframework.web.client.RestTemplate; /** * Test management endpoints against configuration resource. */ +@AutoConfigureMetrics @SpringBootTest(webEnvironment = SpringBootTest.WebEnvironment.RANDOM_PORT, classes = {SpringContextAware.class, GraphAdminApp.class}) @ContextConfiguration(initializers = PropertyPasswordConfiguration.class, classes = {SpringContextAware.class}) @@ -91,14 +94,12 @@ public class MetricsConfigurationTest { headers = new HttpHeaders(); - headers.setAccept(Collections.singletonList(MediaType.APPLICATION_JSON)); - headers.setContentType(MediaType.APPLICATION_JSON); + headers.setAccept(Arrays.asList(MediaType.TEXT_PLAIN)); headers.add("Real-Time", "true"); headers.add("X-FromAppId", "JUNIT"); headers.add("X-TransactionId", "JUNIT"); - String authorization = Base64.getEncoder().encodeToString("AAI:AAI".getBytes("UTF-8")); - headers.add("Authorization", "Basic " + authorization); + headers.setBasicAuth("AAI","AAI"); httpEntity = new HttpEntity(headers); baseUrl = "http://localhost:" + randomPort; @@ -108,15 +109,16 @@ public class MetricsConfigurationTest { @Test public void testManagementEndpointConfiguration() { - ResponseEntity responseEntity = null; + ResponseEntity responseEntity = null; String responseBody = null; //set Accept as text/plain in order to get access of endpoint "/actuator/prometheus" headers.set("Accept", "text/plain"); + headers.setAccept(Arrays.asList(MediaType.TEXT_PLAIN)); httpEntity = new HttpEntity(headers); responseEntity = restTemplate.exchange(actuatorUrl + "/actuator/prometheus", HttpMethod.GET, httpEntity, String.class); - responseBody = (String) responseEntity.getBody(); + responseBody = responseEntity.getBody(); assertEquals(HttpStatus.OK, responseEntity.getStatusCode()); //Set Accept as MediaType.APPLICATION_JSON in order to get access of endpoint "/actuator/info" and "/actuator/health" @@ -124,7 +126,7 @@ public class MetricsConfigurationTest { httpEntity = new HttpEntity(headers); responseEntity = restTemplate.exchange(actuatorUrl + "/actuator/info", HttpMethod.GET, httpEntity, String.class); - responseBody = (String) responseEntity.getBody(); + responseBody = responseEntity.getBody(); assertEquals(HttpStatus.OK, responseEntity.getStatusCode()); responseEntity = diff --git a/src/test/java/org/onap/aai/audit/AuditGraphson2SqlTest.java b/src/test/java/org/onap/aai/audit/AuditGraphson2SqlTest.java index 2bc6250..aa8236e 100644 --- a/src/test/java/org/onap/aai/audit/AuditGraphson2SqlTest.java +++ b/src/test/java/org/onap/aai/audit/AuditGraphson2SqlTest.java @@ -22,22 +22,18 @@ package org.onap.aai.audit; import com.google.gson.JsonObject; import org.junit.Before; -import org.junit.FixMethodOrder; import org.junit.Ignore; import org.junit.Test; -import org.junit.runners.MethodSorters; import org.mockito.Mock; -import static org.mockito.Matchers.*; -import static org.mockito.Mockito.mock; +import static org.mockito.ArgumentMatchers.anyString; +import static org.mockito.ArgumentMatchers.anyLong; import static org.mockito.Mockito.when; import org.onap.aai.AAISetup; import org.onap.aai.edges.EdgeIngestor; import org.onap.aai.edges.exceptions.EdgeRuleNotFoundException; -import org.onap.aai.exceptions.AAIException; import org.onap.aai.rest.client.ApertureService; import org.springframework.beans.factory.annotation.Autowired; -import org.springframework.test.context.TestPropertySource; import java.io.IOException; import java.util.HashMap; diff --git a/src/test/java/org/onap/aai/datasnapshot/DataSnapshot4HistInitTest.java b/src/test/java/org/onap/aai/datasnapshot/DataSnapshot4HistInitTest.java index c297f78..d3fbec7 100644 --- a/src/test/java/org/onap/aai/datasnapshot/DataSnapshot4HistInitTest.java +++ b/src/test/java/org/onap/aai/datasnapshot/DataSnapshot4HistInitTest.java @@ -35,7 +35,8 @@ import org.junit.Test; import org.onap.aai.AAISetup; import org.onap.aai.dbmap.AAIGraph; import org.onap.aai.exceptions.AAIException; -import org.springframework.boot.test.rule.OutputCapture; +import org.springframework.boot.test.system.OutputCaptureRule; + import java.io.File; import java.io.IOException; import java.util.ArrayList; @@ -66,7 +67,7 @@ public class DataSnapshot4HistInitTest extends AAISetup { private DataSnapshot4HistInit dataSnapshot4HistInit; @Rule - public OutputCapture outputCapture = new OutputCapture(); + public OutputCaptureRule outputCapture = new OutputCaptureRule(); @Before public void setup() throws AAIException { diff --git a/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest.java b/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest.java index 7f511e6..9d74651 100644 --- a/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest.java +++ b/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest.java @@ -33,14 +33,8 @@ import org.junit.Test; import org.onap.aai.AAISetup; import org.onap.aai.dbmap.AAIGraph; import org.onap.aai.exceptions.AAIException; -import org.onap.aai.util.AAIConstants; -import org.onap.aai.util.FormatDate; -import org.onap.aai.util.GraphAdminDBUtils; -import org.springframework.boot.test.rule.OutputCapture; +import org.springframework.boot.test.system.OutputCaptureRule; -import com.beust.jcommander.ParameterException; - -import java.lang.NumberFormatException; import java.io.File; import java.io.IOException; import java.nio.file.Files; @@ -66,7 +60,7 @@ public class DataSnapshotTest extends AAISetup { private static final int DELAYSINGLETHREADTEST = 90; @Rule - public OutputCapture outputCapture = new OutputCapture(); + public OutputCaptureRule outputCapture = new OutputCaptureRule(); @Before public void setup() throws AAIException { diff --git a/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest4HistInit.java b/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest4HistInit.java index adcde4e..893a197 100644 --- a/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest4HistInit.java +++ b/src/test/java/org/onap/aai/datasnapshot/DataSnapshotTest4HistInit.java @@ -37,13 +37,8 @@ import org.onap.aai.datagrooming.DataGrooming; import org.onap.aai.dbmap.AAIGraph; import org.onap.aai.exceptions.AAIException; -import org.onap.aai.logging.LogFormatTools; -import org.onap.aai.util.AAISystemExitUtil; -import org.springframework.boot.test.rule.OutputCapture; +import org.springframework.boot.test.system.OutputCaptureRule; -import com.beust.jcommander.ParameterException; - -import java.lang.NumberFormatException; import java.io.File; import java.io.IOException; import java.nio.file.Files; @@ -73,7 +68,7 @@ public class DataSnapshotTest4HistInit extends AAISetup { private DataSnapshot4HistInit dataSnapshot4HistInit; @Rule - public OutputCapture outputCapture = new OutputCapture(); + public OutputCaptureRule outputCapture = new OutputCaptureRule(); @Before public void setup() throws AAIException { diff --git a/src/test/java/org/onap/aai/dbgen/schemamod/SchemaMod4HistTest.java b/src/test/java/org/onap/aai/dbgen/schemamod/SchemaMod4HistTest.java index 8c79cc1..b6d12cd 100644 --- a/src/test/java/org/onap/aai/dbgen/schemamod/SchemaMod4HistTest.java +++ b/src/test/java/org/onap/aai/dbgen/schemamod/SchemaMod4HistTest.java @@ -23,32 +23,15 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.janusgraph.core.JanusGraphTransaction; import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversalSource; -import org.apache.tinkerpop.gremlin.structure.Edge; -import org.apache.tinkerpop.gremlin.structure.Vertex; import org.junit.After; import org.junit.Before; import org.junit.FixMethodOrder; import org.junit.Test; import org.junit.runners.MethodSorters; -import org.mockito.Mock; -import org.mockito.Mockito; import org.onap.aai.AAISetup; import org.onap.aai.dbmap.AAIGraph; import org.onap.aai.exceptions.AAIException; -import org.onap.aai.introspection.LoaderFactory; -import org.onap.aai.setup.SchemaVersions; -import org.springframework.context.annotation.AnnotationConfigApplicationContext; -import static org.mockito.Mockito.when; -import java.io.ByteArrayInputStream; -import java.io.InputStream; -import java.util.ArrayList; -import java.util.HashMap; -import java.util.HashSet; -import java.util.List; - -import static org.hamcrest.CoreMatchers.is; -import static org.hamcrest.MatcherAssert.assertThat; import static org.junit.Assert.*; @FixMethodOrder(MethodSorters.NAME_ASCENDING) diff --git a/src/test/java/org/onap/aai/dbgen/schemamod/SchemaModTest.java b/src/test/java/org/onap/aai/dbgen/schemamod/SchemaModTest.java index caccccf..e66cca3 100644 --- a/src/test/java/org/onap/aai/dbgen/schemamod/SchemaModTest.java +++ b/src/test/java/org/onap/aai/dbgen/schemamod/SchemaModTest.java @@ -23,32 +23,15 @@ import org.slf4j.Logger; import org.slf4j.LoggerFactory; import org.janusgraph.core.JanusGraphTransaction; import org.apache.tinkerpop.gremlin.process.traversal.dsl.graph.GraphTraversalSource; -import org.apache.tinkerpop.gremlin.structure.Edge; import org.apache.tinkerpop.gremlin.structure.Vertex; import org.junit.After; import org.junit.Before; import org.junit.FixMethodOrder; import org.junit.Test; import org.junit.runners.MethodSorters; -import org.mockito.Mock; -import org.mockito.Mockito; import org.onap.aai.AAISetup; import org.onap.aai.dbmap.AAIGraph; import org.onap.aai.exceptions.AAIException; -import org.onap.aai.introspection.LoaderFactory; -import org.onap.aai.setup.SchemaVersions; -import org.springframework.context.annotation.AnnotationConfigApplicationContext; -import static org.mockito.Mockito.when; - -import java.io.ByteArrayInputStream; -import java.io.InputStream; -import java.util.ArrayList; -import java.util.HashMap; -import java.util.HashSet; -import java.util.List; - -import static org.hamcrest.CoreMatchers.is; -import static org.hamcrest.MatcherAssert.assertThat; import static org.junit.Assert.*; @FixMethodOrder(MethodSorters.NAME_ASCENDING) diff --git a/src/test/java/org/onap/aai/migration/v20/MigrateL2DefaultToFalseTest.java b/src/test/java/org/onap/aai/migration/v20/MigrateL2DefaultToFalseTest.java index cba5c45..94413f8 100644 --- a/src/test/java/org/onap/aai/migration/v20/MigrateL2DefaultToFalseTest.java +++ b/src/test/java/org/onap/aai/migration/v20/MigrateL2DefaultToFalseTest.java @@ -33,7 +33,7 @@ import org.onap.aai.serialization.engines.QueryStyle; import org.onap.aai.serialization.engines.TransactionalGraphEngine; import org.onap.aai.setup.SchemaVersions; -import edu.emory.mathcs.backport.java.util.Arrays; +import java.util.Arrays; import static org.junit.Assert.assertTrue; import static org.mockito.Mockito.spy; diff --git a/version.properties b/version.properties index 60c0073..c5bbf54 100644 --- a/version.properties +++ b/version.properties @@ -4,8 +4,8 @@ # because they are used in Jenkins, whose plug-in doesn't support major_version=1 -minor_version=12 -patch_version=3 +minor_version=13 +patch_version=4 base_version=${major_version}.${minor_version}.${patch_version} -- cgit 1.2.3-korg