aboutsummaryrefslogtreecommitdiffstats
path: root/models-interactions
diff options
context:
space:
mode:
authoradheli.tavares <adheli.tavares@est.tech>2023-09-28 18:26:22 +0100
committeradheli.tavares <adheli.tavares@est.tech>2023-09-29 14:44:49 +0100
commit865100d2bbaafac77f1d9e637bca81adc03c4c44 (patch)
tree2aeafd442bf49dd31bd682acbe6f70b3b0dd9bc6 /models-interactions
parent938005505883cf7a636a8840e20e3dc8a0ad9176 (diff)
Fix nexus and sonar vulnerabilities
- IQ nexus dependencies - sonar hotspots and code smells Issue-ID: POLICY-4760 Change-Id: Ic4260c568f9d9164a33aacb34a5ff3a6f91b1bd8 Signed-off-by: adheli.tavares <adheli.tavares@est.tech>
Diffstat (limited to 'models-interactions')
-rw-r--r--models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java24
-rw-r--r--models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java3
-rw-r--r--models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java16
-rw-r--r--models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java18
-rw-r--r--models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java24
-rw-r--r--models-interactions/model-impl/aai/pom.xml2
-rw-r--r--models-interactions/model-impl/cds/pom.xml2
7 files changed, 38 insertions, 51 deletions
diff --git a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java
index b8ec634e3..ffdbde831 100644
--- a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java
+++ b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/AppcOperation.java
@@ -3,6 +3,7 @@
* ONAP
* ================================================================================
* Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2023 Nordix Foundation.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -74,11 +75,10 @@ public abstract class AppcOperation extends BidirectionalTopicOperation<Request,
* Makes a request, given the target VNF. This is a support function for
* {@link #makeRequest(int)}.
*
- * @param attempt attempt number
* @param targetVnf target VNF
* @return a new request
*/
- protected Request makeRequest(int attempt, GenericVnf targetVnf) {
+ protected Request makeRequest(GenericVnf targetVnf) {
var request = new Request();
request.setCommonHeader(new CommonHeader());
request.getCommonHeader().setRequestId(params.getRequestId());
@@ -145,19 +145,13 @@ public abstract class AppcOperation extends BidirectionalTopicOperation<Request,
"unknown APPC-C response status code: " + response.getStatus().getCode());
}
- switch (code) {
- case SUCCESS:
- return Status.SUCCESS;
- case FAILURE:
- return Status.FAILURE;
- case ERROR:
- case REJECT:
- throw new IllegalArgumentException("APP-C request was not accepted, code=" + code);
- case ACCEPT:
- default:
- // awaiting a "final" response
- return Status.STILL_WAITING;
- }
+ return switch (code) {
+ case SUCCESS -> Status.SUCCESS;
+ case FAILURE -> Status.FAILURE;
+ case ERROR, REJECT -> throw new IllegalArgumentException("APP-C request was not accepted, code=" + code);
+ // awaiting a "final" response
+ default -> Status.STILL_WAITING;
+ };
}
/**
diff --git a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java
index 524c797b6..0cf004935 100644
--- a/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java
+++ b/models-interactions/model-actors/actor.appc/src/main/java/org/onap/policy/controlloop/actor/appc/ModifyConfigOperation.java
@@ -3,6 +3,7 @@
* ONAP
* ================================================================================
* Copyright (C) 2020 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2023 Nordix Foundation.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -43,6 +44,6 @@ public class ModifyConfigOperation extends AppcOperation {
@Override
protected Request makeRequest(int attempt) {
- return makeRequest(attempt, getRequiredProperty(OperationProperties.AAI_RESOURCE_VNF, "resource VNF"));
+ return makeRequest(getRequiredProperty(OperationProperties.AAI_RESOURCE_VNF, "resource VNF"));
}
}
diff --git a/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java b/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java
index d40fa025d..b6e9b34e7 100644
--- a/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java
+++ b/models-interactions/model-actors/actor.appc/src/test/java/org/onap/policy/controlloop/actor/appc/AppcOperationTest.java
@@ -90,7 +90,7 @@ public class AppcOperationTest extends BasicAppcOperation {
String subreq = oper.getSubRequestId();
assertNotNull(subreq);
- Request request = oper.makeRequest(2, genvnf);
+ Request request = oper.makeRequest(genvnf);
assertEquals(DEFAULT_OPERATION, request.getAction());
assertNotNull(request.getPayload());
@@ -101,18 +101,18 @@ public class AppcOperationTest extends BasicAppcOperation {
assertEquals(subreq, header.getSubRequestId());
- request = oper.makeRequest(2, genvnf);
+ request = oper.makeRequest(genvnf);
assertEquals(subreq, request.getCommonHeader().getSubRequestId());
// repeat using a null payload
params = params.toBuilder().payload(null).build();
oper = new MyOper(params, config);
- assertEquals(Map.of(AppcOperation.VNF_ID_KEY, MY_VNF), oper.makeRequest(2, genvnf).getPayload());
+ assertEquals(Map.of(AppcOperation.VNF_ID_KEY, MY_VNF), oper.makeRequest(genvnf).getPayload());
}
@Test
public void testConvertPayload() {
- Request request = oper.makeRequest(2, genvnf);
+ Request request = oper.makeRequest(genvnf);
// @formatter:off
assertEquals(
@@ -132,7 +132,7 @@ public class AppcOperationTest extends BasicAppcOperation {
params = params.toBuilder().payload(payload).build();
oper = new MyOper(params, config);
- request = oper.makeRequest(2, genvnf);
+ request = oper.makeRequest(genvnf);
// @formatter:off
assertEquals(
@@ -153,7 +153,7 @@ public class AppcOperationTest extends BasicAppcOperation {
params = params.toBuilder().payload(payload).build();
oper = new MyOper(params, config);
- request = oper.makeRequest(2, genvnf);
+ request = oper.makeRequest(genvnf);
payload.put(AppcOperation.VNF_ID_KEY, MY_VNF);
payload.put(KEY1, "abc");
@@ -166,7 +166,7 @@ public class AppcOperationTest extends BasicAppcOperation {
@Test
public void testGetExpectedKeyValues() {
oper.generateSubRequestId(2);
- Request request = oper.makeRequest(2, genvnf);
+ Request request = oper.makeRequest(genvnf);
assertEquals(Arrays.asList(request.getCommonHeader().getSubRequestId()),
oper.getExpectedKeyValues(50, request));
}
@@ -239,7 +239,7 @@ public class AppcOperationTest extends BasicAppcOperation {
@Override
protected Request makeRequest(int attempt) {
- return makeRequest(attempt, genvnf);
+ return makeRequest(genvnf);
}
}
}
diff --git a/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java b/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java
index 5d256e8ae..4e8f59008 100644
--- a/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java
+++ b/models-interactions/model-actors/actor.appclcm/src/main/java/org/onap/policy/controlloop/actor/appclcm/AppcLcmOperation.java
@@ -3,6 +3,7 @@
* ONAP
* ================================================================================
* Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2023 Nordix Foundation.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -150,18 +151,13 @@ public class AppcLcmOperation extends BidirectionalTopicOperation<AppcLcmDmaapWr
throw new IllegalArgumentException("unknown APPC-LCM response status code: " + status.getCode());
}
- switch (code) {
- case AppcLcmResponseCode.SUCCESS:
- return Status.SUCCESS;
- case AppcLcmResponseCode.FAILURE:
- return Status.FAILURE;
- case AppcLcmResponseCode.ERROR:
- case AppcLcmResponseCode.REJECT:
+ return switch (code) {
+ case AppcLcmResponseCode.SUCCESS -> Status.SUCCESS;
+ case AppcLcmResponseCode.FAILURE -> Status.FAILURE;
+ case AppcLcmResponseCode.ERROR, AppcLcmResponseCode.REJECT ->
throw new IllegalArgumentException("APPC-LCM request was not accepted, code=" + code);
- case AppcLcmResponseCode.ACCEPTED:
- default:
- return Status.STILL_WAITING;
- }
+ default -> Status.STILL_WAITING;
+ };
}
/**
diff --git a/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java b/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java
index 5b30c673a..7e57bcccd 100644
--- a/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java
+++ b/models-interactions/model-actors/actor.sdnr/src/main/java/org/onap/policy/controlloop/actor/sdnr/SdnrOperation.java
@@ -3,6 +3,7 @@
* SdnrOperation
* ================================================================================
* Copyright (C) 2020-2021 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2023 Nordix Foundation.
* ================================================================================
* Licensed under the Apache License, Version 2.0 (the "License");
* you may not use this file except in compliance with the License.
@@ -90,22 +91,17 @@ public class SdnrOperation extends BidirectionalTopicOperation<PciMessage, PciMe
return Status.FAILURE;
}
- switch (code) {
- case SUCCESS:
- case PARTIAL_SUCCESS:
- return Status.SUCCESS;
- case FAILURE:
- case PARTIAL_FAILURE:
- return Status.FAILURE;
- case ERROR:
- case REJECT:
+ return switch (code) {
+ case SUCCESS, PARTIAL_SUCCESS -> Status.SUCCESS;
+ case FAILURE, PARTIAL_FAILURE -> Status.FAILURE;
+ case ERROR, REJECT -> {
logger.warn("SDNR request was not accepted, code={}", code);
- return Status.FAILURE;
- case ACCEPTED:
- default:
+ yield Status.FAILURE;
+ }
+ default ->
// awaiting a "final" response
- return Status.STILL_WAITING;
- }
+ Status.STILL_WAITING;
+ };
}
/**
diff --git a/models-interactions/model-impl/aai/pom.xml b/models-interactions/model-impl/aai/pom.xml
index cc9619a69..fd3707b35 100644
--- a/models-interactions/model-impl/aai/pom.xml
+++ b/models-interactions/model-impl/aai/pom.xml
@@ -52,7 +52,7 @@
<dependency>
<groupId>org.onap.aai.schema-service</groupId>
<artifactId>aai-schema</artifactId>
- <version>1.7.2</version>
+ <version>1.9.6</version>
<exclusions>
<exclusion>
<groupId>com.google.code.javaparser</groupId>
diff --git a/models-interactions/model-impl/cds/pom.xml b/models-interactions/model-impl/cds/pom.xml
index d70265426..1bfdc7d7f 100644
--- a/models-interactions/model-impl/cds/pom.xml
+++ b/models-interactions/model-impl/cds/pom.xml
@@ -35,7 +35,7 @@
</description>
<properties>
- <grpc.version>1.25.0</grpc.version>
+ <grpc.version>1.29.0</grpc.version>
</properties>
<dependencies>