[POLICY-134] Fixed bug in PolicyLogger.

PolicyLogger.info(Object arg0) calls itself recursively until it eventually
causes a stack overflow error.

Fix is to call debugLogger.info(MessageCodes, String) from within
PolicyLogger.info(Object).

Additionally, PolicyLogger.info(String arg0) was removed as
debugLogger.info(MessageCodes, String) can be called directly.

Change-Id: Id141027b544efa06b79f54a6726843b2a47481b9
Signed-off-by: Magnusen, Drew (dm741q) <dm741q@att.com>

diff --git a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java
index d0cc9c1..7507c9e 100644 (file)
--- a/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java
+++ b/common-logging/src/main/java/org/onap/policy/common/logging/eelf/PolicyLogger.java
@@ -366,14 +366,6 @@ public class PolicyLogger {
                debugLogger.info(MessageCodes.GENERAL_INFO, arg0);
        }
 
-       /**
-        * Records only one String message without its class name passed in
-        * @param arg0
-        */
-       public static void info(String arg0) {
-               MDC.put(CLASS_NAME, "");
-               debugLogger.info(MessageCodes.GENERAL_INFO, arg0);
-       }
        
        /**
         * Records only one String message
@@ -381,7 +373,7 @@ public class PolicyLogger {
         */
        public static void info(Object arg0) {
                MDC.put(CLASS_NAME, "");
-               info(arg0);
+               debugLogger.info(MessageCodes.GENERAL_INFO, String.valueOf(arg0));
        }
 
        /**