From 9d3f4f0bef642836731213dc05e9127bca34200c Mon Sep 17 00:00:00 2001 From: Pamela Dragosh Date: Wed, 23 Jun 2021 11:05:05 -0400 Subject: [PATCH] Use protected methods vs private Application and common code in xacml-pdp is meant to be available for developers to re-use and/or extend as desired. Private methods are unnecessary at this time, but could be re-visited in a later release. For Honolulu, simply moving to protected to keep functionality the same. Possibly for Istanbul we can make more changes to support mixing of ONAP application with custom applications during runtime. Issue-ID: POLICY-3305 Change-Id: I6db23bda3c81fdbbf86e39cc2cd2e706d8bee63d Signed-off-by: Pamela Dragosh (cherry picked from commit 8e0aa0ffa5cef537de1c841acf38300832c52a56) Signed-off-by: Pamela Dragosh --- .../xacml/application/common/std/StdCombinedPolicyRequest.java | 5 +++-- .../xacml/application/common/std/StdMatchableTranslator.java | 8 ++++---- .../policy/xacml/pdp/application/guard/GuardPolicyRequest.java | 4 ++-- .../policy/xacml/pdp/application/guard/GuardTranslator.java | 10 +++++----- .../pdp/application/monitoring/MonitoringPdpApplication.java | 2 +- .../pdp/application/nativ/NativePdpApplicationTranslator.java | 2 +- .../application/optimization/OptimizationPdpApplication.java | 8 ++++---- .../optimization/OptimizationPdpApplicationTranslator.java | 8 ++++---- 8 files changed, 24 insertions(+), 23 deletions(-) diff --git a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdCombinedPolicyRequest.java b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdCombinedPolicyRequest.java index 75b89d24..9618d5b7 100644 --- a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdCombinedPolicyRequest.java +++ b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdCombinedPolicyRequest.java @@ -112,14 +112,15 @@ public class StdCombinedPolicyRequest { return request; } - private static StdCombinedPolicyRequest addPolicyIds(StdCombinedPolicyRequest request, Collection ids) { + protected static StdCombinedPolicyRequest addPolicyIds(StdCombinedPolicyRequest request, Collection ids) { for (Object id : ids) { request.resource.add(id.toString()); } return request; } - private static StdCombinedPolicyRequest addPolicyTypes(StdCombinedPolicyRequest request, Collection types) { + protected static StdCombinedPolicyRequest addPolicyTypes(StdCombinedPolicyRequest request, + Collection types) { for (Object type : types) { request.resourcePolicyType.add(type.toString()); } diff --git a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java index 9d0124bb..8d712768 100644 --- a/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java +++ b/applications/common/src/main/java/org/onap/policy/pdp/xacml/application/common/std/StdMatchableTranslator.java @@ -477,7 +477,7 @@ public class StdMatchableTranslator extends StdBaseTranslator implements Matcha * @param policyTypeId ToscaConceptIdentifier to find * @return ToscaPolicyType object. Can be null if failure. */ - private ToscaServiceTemplate findPolicyType(ToscaConceptIdentifier policyTypeId) { + protected ToscaServiceTemplate findPolicyType(ToscaConceptIdentifier policyTypeId) { // // Is it loaded in memory? // @@ -508,7 +508,7 @@ public class StdMatchableTranslator extends StdBaseTranslator implements Matcha * @param policyTypeId ToscaConceptIdentifier input * @return ToscaPolicyType object. Null if failure. */ - private ToscaServiceTemplate loadPolicyType(ToscaConceptIdentifier policyTypeId) { + protected ToscaServiceTemplate loadPolicyType(ToscaConceptIdentifier policyTypeId) { // // Construct what the file name should be // @@ -567,7 +567,7 @@ public class StdMatchableTranslator extends StdBaseTranslator implements Matcha * @param policyTypePath Path object to store locally * @return ToscaPolicyType object. Null if failure. */ - private synchronized ToscaServiceTemplate pullPolicyType(ToscaConceptIdentifier policyTypeId, + protected synchronized ToscaServiceTemplate pullPolicyType(ToscaConceptIdentifier policyTypeId, Path policyTypePath) { // // This is what we return @@ -603,7 +603,7 @@ public class StdMatchableTranslator extends StdBaseTranslator implements Matcha * @param policyTypeId ToscaConceptIdentifier * @return Path object */ - private Path constructLocalFilePath(ToscaConceptIdentifier policyTypeId) { + protected Path constructLocalFilePath(ToscaConceptIdentifier policyTypeId) { return Paths.get(this.pathForData.toAbsolutePath().toString(), policyTypeId.getName() + "-" + policyTypeId.getVersion() + ".yaml"); } diff --git a/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardPolicyRequest.java b/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardPolicyRequest.java index 96b2878a..8a77449c 100644 --- a/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardPolicyRequest.java +++ b/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardPolicyRequest.java @@ -167,7 +167,7 @@ public class GuardPolicyRequest { return request; } - private static GuardPolicyRequest findFields(GuardPolicyRequest request, Map guard) + protected static GuardPolicyRequest findFields(GuardPolicyRequest request, Map guard) throws ToscaPolicyConversionException { if (guard.containsKey("actor")) { request.actorId = guard.get("actor").toString(); @@ -192,7 +192,7 @@ public class GuardPolicyRequest { return request; } - private static GuardPolicyRequest findFilterFields(GuardPolicyRequest request, Map guard) { + protected static GuardPolicyRequest findFilterFields(GuardPolicyRequest request, Map guard) { if (guard.containsKey("generic-vnf.vnf-name")) { request.vnfName = guard.get("generic-vnf.vnf-name").toString(); } diff --git a/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardTranslator.java b/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardTranslator.java index d606cc2e..df5fe43e 100644 --- a/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardTranslator.java +++ b/applications/guard/src/main/java/org/onap/policy/xacml/pdp/application/guard/GuardTranslator.java @@ -194,7 +194,7 @@ public class GuardTranslator implements ToscaPolicyTranslator { * @param variable VariableDefinitionType to add * @param newPolicyType PolicyType that will be updated */ - private void addVariableToConditionTypes(VariableReferenceType variable, + protected void addVariableToConditionTypes(VariableReferenceType variable, PolicyType newPolicyType) { // // Iterate through the rules @@ -432,7 +432,7 @@ public class GuardTranslator implements ToscaPolicyTranslator { return reference; } - private Object parseTimestamp(String string) throws ToscaPolicyConversionException { + protected Object parseTimestamp(String string) throws ToscaPolicyConversionException { // // First see if it is a full datetime object // @@ -675,7 +675,7 @@ public class GuardTranslator implements ToscaPolicyTranslator { } } - private String validateFilterPropertyField(String field) + protected String validateFilterPropertyField(String field) throws ToscaPolicyConversionException { String fieldLowerCase = field.toLowerCase(); switch (fieldLowerCase) { @@ -691,7 +691,7 @@ public class GuardTranslator implements ToscaPolicyTranslator { } } - private Identifier validateFilterPropertyFunction(String function) + protected Identifier validateFilterPropertyFunction(String function) throws ToscaPolicyConversionException { switch (function.toLowerCase()) { case "string-equal": @@ -719,7 +719,7 @@ public class GuardTranslator implements ToscaPolicyTranslator { } } - private RuleType createFilterRule(String ruleId, String field, String filter, Identifier function, + protected RuleType createFilterRule(String ruleId, String field, String filter, Identifier function, boolean isBlacklisted) { var rule = new RuleType(); rule.setRuleId(ruleId); diff --git a/applications/monitoring/src/main/java/org/onap/policy/xacml/pdp/application/monitoring/MonitoringPdpApplication.java b/applications/monitoring/src/main/java/org/onap/policy/xacml/pdp/application/monitoring/MonitoringPdpApplication.java index fc03f1c4..95150c37 100644 --- a/applications/monitoring/src/main/java/org/onap/policy/xacml/pdp/application/monitoring/MonitoringPdpApplication.java +++ b/applications/monitoring/src/main/java/org/onap/policy/xacml/pdp/application/monitoring/MonitoringPdpApplication.java @@ -118,7 +118,7 @@ public class MonitoringPdpApplication extends StdXacmlApplicationServiceProvider * * @param queryParams - http request query parameters */ - private boolean checkAbbreviateResults(Map queryParams) { + protected boolean checkAbbreviateResults(Map queryParams) { if (queryParams != null && !queryParams.isEmpty()) { // Check if query params contains "abbrev" flag if (queryParams.containsKey("abbrev")) { diff --git a/applications/native/src/main/java/org/onap/policy/xacml/pdp/application/nativ/NativePdpApplicationTranslator.java b/applications/native/src/main/java/org/onap/policy/xacml/pdp/application/nativ/NativePdpApplicationTranslator.java index a1230c2d..06e324d2 100644 --- a/applications/native/src/main/java/org/onap/policy/xacml/pdp/application/nativ/NativePdpApplicationTranslator.java +++ b/applications/native/src/main/java/org/onap/policy/xacml/pdp/application/nativ/NativePdpApplicationTranslator.java @@ -86,7 +86,7 @@ public class NativePdpApplicationTranslator implements ToscaPolicyTranslator { } } - private String getNativeXacmlPolicy(ToscaPolicy toscaPolicy) throws ToscaPolicyConversionException { + protected String getNativeXacmlPolicy(ToscaPolicy toscaPolicy) throws ToscaPolicyConversionException { var nativeDefinition = ToscaPolicyTranslatorUtils.decodeProperties(toscaPolicy.getProperties(), NativeDefinition.class); diff --git a/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplication.java b/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplication.java index 92a8e688..998ed537 100644 --- a/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplication.java +++ b/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplication.java @@ -188,14 +188,14 @@ public class OptimizationPdpApplication extends StdXacmlApplicationServiceProvid } @SuppressWarnings("unchecked") - private boolean hasSubscriberAttributes(DecisionRequest request) { + protected boolean hasSubscriberAttributes(DecisionRequest request) { return request.getContext() != null && request.getContext().containsKey(RESOURCE_SUBSCRIBERNAME) && request.getContext().get(RESOURCE_SUBSCRIBERNAME) instanceof List && ! ((List) request.getContext().get(RESOURCE_SUBSCRIBERNAME)).isEmpty(); } - private boolean addSubscriberAttributes(Response xacmlResponse, DecisionRequest initialRequest) { + protected boolean addSubscriberAttributes(Response xacmlResponse, DecisionRequest initialRequest) { // // This has multiple results right now because of how the attributes were added to the // request. That will have to be fixed in the future, for now find the Permit result @@ -221,7 +221,7 @@ public class OptimizationPdpApplication extends StdXacmlApplicationServiceProvid return false; } - private void addSubscriberAdvice(Response xacmlResponse, DecisionResponse response) { + protected void addSubscriberAdvice(Response xacmlResponse, DecisionResponse response) { // // Again find the Permit result // @@ -237,7 +237,7 @@ public class OptimizationPdpApplication extends StdXacmlApplicationServiceProvid @SuppressWarnings("unchecked") - private void scanAdvice(Collection adviceCollection, DecisionRequest initialRequest) { + protected void scanAdvice(Collection adviceCollection, DecisionRequest initialRequest) { // // There really should only be one advice object // diff --git a/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplicationTranslator.java b/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplicationTranslator.java index e1fe2dcd..84a6ffae 100644 --- a/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplicationTranslator.java +++ b/applications/optimization/src/main/java/org/onap/policy/xacml/pdp/application/optimization/OptimizationPdpApplicationTranslator.java @@ -136,7 +136,7 @@ public class OptimizationPdpApplicationTranslator extends StdMatchableTranslator } @SuppressWarnings("unchecked") - private static void addValuesToMap(Object values, String key, Map mapAdvice) { + protected static void addValuesToMap(Object values, String key, Map mapAdvice) { if (values instanceof Collection) { List valueList = new ArrayList<>(); ((Collection) values).forEach(val -> valueList.add(val.toString())); @@ -147,7 +147,7 @@ public class OptimizationPdpApplicationTranslator extends StdMatchableTranslator } - private static PolicyType addSubscriberNameIntoTarget(PolicyType policy, + protected static PolicyType addSubscriberNameIntoTarget(PolicyType policy, Map subscriberProperties) throws ToscaPolicyConversionException { // // Iterate through all the subscriber names @@ -174,7 +174,7 @@ public class OptimizationPdpApplicationTranslator extends StdMatchableTranslator return policy; } - private static AdviceExpressionsType generateSubscriberAdvice(Map subscriberProperties) + protected static AdviceExpressionsType generateSubscriberAdvice(Map subscriberProperties) throws ToscaPolicyConversionException { // // Create our subscriber advice expression @@ -207,7 +207,7 @@ public class OptimizationPdpApplicationTranslator extends StdMatchableTranslator return adviceExpressions; } - private static void generateSubscriberAdviceAttributes(AdviceExpressionType adviceExpression, + protected static void generateSubscriberAdviceAttributes(AdviceExpressionType adviceExpression, Identifier attributeId, Collection adviceAttribute) { for (Object attribute : adviceAttribute) { var value = new AttributeValueType(); -- 2.16.6