Use helper class to retrieve network logger 80/77480/3
authorKrzysztof Opasiak <k.opasiak@samsung.com>
Mon, 28 Jan 2019 20:24:13 +0000 (21:24 +0100)
committerKrzysztof Opasiak <k.opasiak@samsung.com>
Sat, 2 Feb 2019 10:15:39 +0000 (11:15 +0100)
Instead of abusing constant from Topic interface use new dedicated
helper class to retrive network logger.

Issue-ID: POLICY-1486
Depends-On: Ic1f07466e1ab6abcc00098e617f0e82da19e852d
Change-Id: I453a14c3dda337f2a4d08914ead32b37adc17353
Signed-off-by: Krzysztof Opasiak <k.opasiak@samsung.com>
controlloop/common/guard/src/main/java/org/onap/policy/guard/PolicyGuardXacmlHelper.java
controlloop/common/model-impl/aai/src/main/java/org/onap/policy/aai/AaiManager.java
controlloop/common/model-impl/sdnc/src/main/java/org/onap/policy/sdnc/SdncManager.java
controlloop/common/model-impl/so/src/main/java/org/onap/policy/so/SoManager.java
controlloop/common/model-impl/vfc/src/main/java/org/onap/policy/vfc/VfcManager.java
controlloop/templates/template.demo.clc/src/main/java/org/onap/policy/guard/PolicyGuardXacmlHelperEmbedded.java

index c2dfc90..f9a01a7 100644 (file)
@@ -3,13 +3,14 @@
  * guard
  * ================================================================================
  * Copyright (C) 2017-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -41,6 +42,7 @@ import java.util.UUID;
 import org.apache.commons.io.IOUtils;
 import org.apache.http.entity.ContentType;
 import org.json.JSONObject;
+import org.onap.policy.common.utils.slf4j.LoggerFactoryWrapper;
 import org.onap.policy.drools.system.PolicyEngine;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -48,8 +50,7 @@ import org.slf4j.LoggerFactory;
 
 public class PolicyGuardXacmlHelper {
     private static final Logger logger = LoggerFactory.getLogger(PolicyGuardXacmlHelper.class);
-    private static final Logger netLogger =
-            LoggerFactory.getLogger(org.onap.policy.common.endpoints.event.comm.Topic.NETWORK_LOGGER);
+    private static final Logger netLogger = LoggerFactoryWrapper.getNetworkLogger();
 
     // Constant for the systme line separator
     private static final String SYSTEM_LS = System.lineSeparator();
@@ -80,7 +81,7 @@ public class PolicyGuardXacmlHelper {
 
     /**
      * Call PDP.
-     * 
+     *
      * @param xacmlReq the XACML request
      * @return the response
      */
@@ -127,7 +128,7 @@ public class PolicyGuardXacmlHelper {
 
     /**
      * This makes an HTTP POST call to a running PDP RESTful servlet to get a decision.
-     * 
+     *
      * @param is the InputStream
      * @param authorization the Authorization
      * @param clientauth the ClientAuth
@@ -217,7 +218,7 @@ public class PolicyGuardXacmlHelper {
 
     /**
      * Parse XACML PDP response.
-     * 
+     *
      * @param xacmlResponse the XACML response
      * @return the PolicyGuardResponse
      */
index 96d606a..6136aee 100644 (file)
@@ -3,13 +3,14 @@
  * aai
  * ================================================================================
  * Copyright (C) 2017, 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -27,6 +28,7 @@ import java.util.Map;
 import java.util.UUID;
 
 import org.onap.policy.aai.util.Serialization;
+import org.onap.policy.common.utils.slf4j.LoggerFactoryWrapper;
 import org.onap.policy.rest.RestManager;
 import org.onap.policy.rest.RestManager.Pair;
 import org.slf4j.Logger;
@@ -44,8 +46,7 @@ public final class AaiManager {
     private static final Logger logger = LoggerFactory.getLogger(AaiManager.class);
 
     /** The Constant netLogger. */
-    private static final Logger netLogger =
-            LoggerFactory.getLogger(org.onap.policy.common.endpoints.event.comm.Topic.NETWORK_LOGGER);
+    private static final Logger netLogger = LoggerFactoryWrapper.getNetworkLogger();
 
     /** The rest manager. */
     // The REST manager used for processing REST calls for this AAI manager
index 2e39cd9..ae43200 100644 (file)
@@ -3,6 +3,7 @@
  * Copyright (C) 2018 Huawei. All rights reserved.
  * ================================================================================
  * Modifications Copyright (C) 2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -27,6 +28,7 @@ import java.util.HashMap;
 import java.util.Map;
 
 import org.drools.core.WorkingMemory;
+import org.onap.policy.common.utils.slf4j.LoggerFactoryWrapper;
 import org.onap.policy.drools.system.PolicyEngine;
 import org.onap.policy.rest.RestManager;
 import org.onap.policy.rest.RestManager.Pair;
@@ -34,7 +36,7 @@ import org.onap.policy.sdnc.util.Serialization;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
-   
+
 public final class SdncManager implements Runnable {
     private static final String SYSTEM_LS = System.lineSeparator();
 
@@ -44,15 +46,14 @@ public final class SdncManager implements Runnable {
     private SdncRequest sdncRequest;
     private WorkingMemory workingMem;
     private static final Logger logger = LoggerFactory.getLogger(SdncManager.class);
-    private static final Logger netLogger = 
-            LoggerFactory.getLogger(org.onap.policy.common.endpoints.event.comm.Topic.NETWORK_LOGGER);
+    private static final Logger netLogger = LoggerFactoryWrapper.getNetworkLogger();
 
     // The REST manager used for processing REST calls for this Sdnc manager
     private RestManager restManager;
-    
+
     /**
      * Constructor.
-     * 
+     *
      * @param wm Drools working memory
      * @param request request
      */
@@ -67,13 +68,13 @@ public final class SdncManager implements Runnable {
 
         restManager = new RestManager();
 
-        setSdncParams(getPeManagerEnvProperty("sdnc.url"), getPeManagerEnvProperty("sdnc.username"), 
+        setSdncParams(getPeManagerEnvProperty("sdnc.url"), getPeManagerEnvProperty("sdnc.username"),
             getPeManagerEnvProperty("sdnc.password"));
     }
 
     /**
      * Set the parameters.
-     * 
+     *
      * @param baseUrl base URL
      * @param name username
      * @param pwd password
@@ -157,7 +158,7 @@ public final class SdncManager implements Runnable {
         String enginePropertyValue = PolicyEngine.manager.getEnvironmentProperty(enginePropertyName);
         if (enginePropertyValue == null) {
             throw new IllegalArgumentException(
-                "The value of policy engine manager environment property \"" 
+                "The value of policy engine manager environment property \""
                    + enginePropertyName + "\" may not be null"
             );
         }
index 20136f7..7c145e0 100644 (file)
@@ -3,6 +3,7 @@
  * so
  * ================================================================================
  * Copyright (C) 2017-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -29,6 +30,7 @@ import java.util.concurrent.ExecutorService;
 import java.util.concurrent.Executors;
 import java.util.concurrent.Future;
 import org.drools.core.WorkingMemory;
+import org.onap.policy.common.utils.slf4j.LoggerFactoryWrapper;
 import org.onap.policy.drools.system.PolicyEngine;
 import org.onap.policy.rest.RestManager;
 import org.onap.policy.rest.RestManager.Pair;
@@ -45,8 +47,8 @@ import org.slf4j.LoggerFactory;
  */
 public final class SoManager {
     private static final Logger logger = LoggerFactory.getLogger(SoManager.class);
-    private static final Logger netLogger =
-                    LoggerFactory.getLogger(org.onap.policy.common.endpoints.event.comm.Topic.NETWORK_LOGGER);
+    private static final Logger netLogger = LoggerFactoryWrapper.getNetworkLogger();
+
     private static ExecutorService executors = Executors.newCachedThreadPool();
 
     private static final int SO_RESPONSE_ERROR = 999;
@@ -121,10 +123,10 @@ public final class SoManager {
      * @param request            the SO request
      * @return a concurrent Future for the thread that handles the request
      */
-    public Future<SoResponse> asyncSoRestCall(final String requestId, 
+    public Future<SoResponse> asyncSoRestCall(final String requestId,
             final WorkingMemory wm,
-            final String serviceInstanceId, 
-            final String vnfInstanceId, 
+            final String serviceInstanceId,
+            final String vnfInstanceId,
             final String vfModuleInstanceId, final SoRequest request) {
         return executors.submit(new AsyncSoRestCallThread(requestId, wm, serviceInstanceId, vnfInstanceId,
                 vfModuleInstanceId, request));
@@ -151,9 +153,9 @@ public final class SoManager {
          * @param vfModuleInstanceId the vf module instance id (not null in case of delete vf module request)
          * @param request            the request itself
          */
-        private AsyncSoRestCallThread(final String requestId, 
+        private AsyncSoRestCallThread(final String requestId,
                 final WorkingMemory wm, final String serviceInstanceId,
-                final String vnfInstanceId, final String vfModuleInstanceId, 
+                final String vnfInstanceId, final String vfModuleInstanceId,
                 final SoRequest request) {
             this.requestId = requestId;
             this.wm = wm;
index 7cca6c5..0acab8b 100644 (file)
@@ -2,6 +2,7 @@
  * ============LICENSE_START=======================================================
  * Copyright (C) 2017-2018 Intel Corp, AT&T. All rights reserved.
  * Modifications Copyright (C) 2018-2019 AT&T Corporation. All rights reserved.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -25,6 +26,7 @@ import java.util.HashMap;
 import java.util.Map;
 
 import org.drools.core.WorkingMemory;
+import org.onap.policy.common.utils.slf4j.LoggerFactoryWrapper;
 import org.onap.policy.drools.system.PolicyEngine;
 import org.onap.policy.rest.RestManager;
 import org.onap.policy.rest.RestManager.Pair;
@@ -41,15 +43,14 @@ public final class VfcManager implements Runnable {
     private VfcRequest vfcRequest;
     private WorkingMemory workingMem;
     private static final Logger logger = LoggerFactory.getLogger(VfcManager.class);
-    private static final Logger netLogger =
-            LoggerFactory.getLogger(org.onap.policy.common.endpoints.event.comm.Topic.NETWORK_LOGGER);
+    private static final Logger netLogger = LoggerFactoryWrapper.getNetworkLogger();
 
     // The REST manager used for processing REST calls for this VFC manager
     private RestManager restManager;
 
     /**
      * Constructor.
-     * 
+     *
      * @param wm Drools working memory
      * @param request request
      */
@@ -70,7 +71,7 @@ public final class VfcManager implements Runnable {
 
     /**
      * Set the parameters.
-     * 
+     *
      * @param baseUrl base URL
      * @param name username
      * @param pwd password
@@ -163,7 +164,7 @@ public final class VfcManager implements Runnable {
 
     /**
      * Protected setter for rest manager to allow mocked rest manager to be used for testing.
-     * 
+     *
      * @param restManager the test REST manager
      */
     protected void setRestManager(final RestManager restManager) {
@@ -173,7 +174,7 @@ public final class VfcManager implements Runnable {
     /**
      * This method reads and validates environmental properties coming from the policy engine. Null
      * properties cause an {@link IllegalArgumentException} runtime exception to be thrown
-     * 
+     *
      * @param string the name of the parameter to retrieve
      * @return the property value
      */
index 1e9f337..075482e 100644 (file)
@@ -3,13 +3,14 @@
  * guard
  * ================================================================================
  * Copyright (C) 2018-2019 AT&T Intellectual Property. All rights reserved.
+ * Modifications Copyright (C) 2019 Samsung Electronics Co., Ltd.
  * ================================================================================
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
  * You may obtain a copy of the License at
- * 
+ *
  *      http://www.apache.org/licenses/LICENSE-2.0
- * 
+ *
  * Unless required by applicable law or agreed to in writing, software
  * distributed under the License is distributed on an "AS IS" BASIS,
  * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
@@ -47,6 +48,7 @@ import java.util.UUID;
 import org.apache.commons.io.IOUtils;
 import org.apache.http.entity.ContentType;
 import org.json.JSONObject;
+import org.onap.policy.common.utils.slf4j.LoggerFactoryWrapper;
 import org.onap.policy.drools.system.PolicyEngine;
 import org.onap.policy.guard.PolicyGuardResponse;
 import org.onap.policy.guard.PolicyGuardXacmlRequestAttributes;
@@ -57,8 +59,7 @@ import org.slf4j.LoggerFactory;
 
 public class PolicyGuardXacmlHelperEmbedded {
     private static final Logger logger = LoggerFactory.getLogger(PolicyGuardXacmlHelperEmbedded.class);
-    private static final Logger netLogger =
-            LoggerFactory.getLogger(org.onap.policy.common.endpoints.event.comm.Topic.NETWORK_LOGGER);
+    private static final Logger netLogger = LoggerFactoryWrapper.getNetworkLogger();
 
     // Constant for the system line separator
     private static final String SYSTEM_LS = System.lineSeparator();
@@ -68,7 +69,7 @@ public class PolicyGuardXacmlHelperEmbedded {
 
     // REST timeout, initialized from 'pdpx.timeout' property
     private int timeout = 20000;
-    
+
     public PolicyGuardXacmlHelperEmbedded() {
         init(PolicyEngine.manager.getEnvironment());
     }
@@ -89,7 +90,7 @@ public class PolicyGuardXacmlHelperEmbedded {
 
     /**
      * Call PDP.
-     * 
+     *
      * @param xacmlReq the XACML request
      * @return the response
      */
@@ -140,7 +141,7 @@ public class PolicyGuardXacmlHelperEmbedded {
 
     /**
      * This makes an HTTP POST call to a running PDP RESTful servlet to get a decision.
-     * 
+     *
      * @param is the InputStream
      * @param authorization the Authorization
      * @param clientauth the ClientAuth
@@ -230,7 +231,7 @@ public class PolicyGuardXacmlHelperEmbedded {
 
     /**
      * Call embedded PDP.
-     * 
+     *
      * @param xacmlReq the XACML request
      * @return the response
      */
@@ -239,7 +240,7 @@ public class PolicyGuardXacmlHelperEmbedded {
         Properties props = new Properties();
         //
         // Get properties
-        // 
+        //
         try ( InputStream is = new FileInputStream(propfile);
               InputStreamReader isr = new InputStreamReader(is);
               BufferedReader br = new BufferedReader(isr) ) {
@@ -285,7 +286,7 @@ public class PolicyGuardXacmlHelperEmbedded {
 
     /**
      * Parse XACML PDP response.
-     * 
+     *
      * @param xacmlResponse the XACML response
      * @return the PolicyGuardResponse
      */
@@ -482,7 +483,7 @@ public class PolicyGuardXacmlHelperEmbedded {
         // if content length is -1, response is chunked, and
         // TCP connection will be dropped at the end
         byte[] buf = new byte[contentLength < 0 ? 1024 : contentLength];
-        
+
         int offset = 0;
         do {
             int size = inputStream.read(buf, offset, buf.length - offset);