Adjust Logging Dir settings for ONAP Users 95/54695/1
authorInstrumental <jonathan.gathman@att.com>
Wed, 13 Jun 2018 22:19:07 +0000 (17:19 -0500)
committerInstrumental <jonathan.gathman@att.com>
Wed, 13 Jun 2018 22:21:44 +0000 (17:21 -0500)
Issue-ID: AAF-360
Change-Id: If19090ef482372d3a3a9aa2856762af91bc54017
Signed-off-by: Instrumental <jonathan.gathman@att.com>
auth/auth-core/src/main/java/org/onap/aaf/auth/server/Log4JLogIt.java
auth/auth-deforg/src/main/java/org/onap/aaf/org/DefaultOrg.java
cadi/aaf/src/test/java/org/onap/aaf/cadi/cm/test/.gitignore [new file with mode: 0644]
misc/log4j/src/main/java/org/onap/aaf/misc/env/log4j/LogFileNamer.java
misc/log4j/src/test/java/org/onap/aaf/misc/env/log4j/JU_LogFileNamerTest.java

index e295c86..e6f2fc9 100644 (file)
@@ -62,6 +62,9 @@ public class Log4JLogIt implements LogIt {
                        logs.mkdirs();
                }
 
+               if(System.getProperty("log4j.configuration")==null) {
+                       System.setProperty("log4j.configuration", etc_dir+'/'+propsFile);
+               }
                LogFileNamer lfn = new LogFileNamer(log_dir,root);
                try {
                        service=lfn.setAppender("service"); // when name is split, i.e. authz|service, the Appender is "authz", and "service"
index 3d42b63..eefb273 100644 (file)
@@ -21,7 +21,8 @@
  ******************************************************************************/
 package org.onap.aaf.org;
 
-import java.io.*;
+import java.io.File;
+import java.io.IOException;
 import java.util.ArrayList;
 import java.util.Date;
 import java.util.GregorianCalendar;
@@ -160,7 +161,6 @@ public class DefaultOrg implements Organization {
        @Override
        public DefaultOrgIdentity getIdentity(AuthzTrans trans, String id) throws OrganizationException {
                int at = id.indexOf('@');
-               String attt = at<0?id:id.substring(0, at);
                return new DefaultOrgIdentity(trans,at<0?id:id.substring(0, at),this);
        }
 
diff --git a/cadi/aaf/src/test/java/org/onap/aaf/cadi/cm/test/.gitignore b/cadi/aaf/src/test/java/org/onap/aaf/cadi/cm/test/.gitignore
new file mode 100644 (file)
index 0000000..52448be
--- /dev/null
@@ -0,0 +1 @@
+/JU_CmAgentCreate.java
index a98ba7c..c6537de 100644 (file)
@@ -59,15 +59,14 @@ public class LogFileNamer {
         * @throws IOException\r
         */\r
        public String setAppender(String appender) throws IOException {\r
-               String filename;\r
                int i = 0;\r
                File f;\r
-               while ((f = new File(filename = String.format(FILE_FORMAT_STR, dir, root, appender, ending, i))).exists()) {\r
+               while ((f = new File(String.format(FILE_FORMAT_STR, dir, root, appender, ending, i))).exists()) {\r
                        ++i;\r
                }\r
-               ;\r
+               \r
                f.createNewFile();\r
-               System.setProperty("LOG4J_FILENAME_" + appender, filename);\r
+               System.setProperty("LOG4J_FILENAME_" + appender, f.getCanonicalPath());\r
                return appender;\r
        }\r
 \r
index 0ee79a5..b96d6dd 100644 (file)
@@ -34,6 +34,7 @@ import org.junit.Before;
 import org.junit.Test;\r
 \r
 public class JU_LogFileNamerTest {\r
+       private File dir = new File(".");\r
 \r
        private String ending = new SimpleDateFormat("YYYYMMdd").format(new Date());\r
 \r
@@ -43,26 +44,26 @@ public class JU_LogFileNamerTest {
 \r
        @Test\r
        public void test() throws IOException {\r
-               LogFileNamer logFileNamer = new LogFileNamer(".", "log");\r
+               LogFileNamer logFileNamer = new LogFileNamer(dir.getCanonicalPath(), "log");\r
                assertEquals(logFileNamer, logFileNamer.noPID());\r
 \r
                logFileNamer.setAppender("Append");\r
-               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), "./log-Append" + ending + "_0.log");\r
+               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), dir.getCanonicalFile()+"/log-Append" + ending + "_0.log");\r
 \r
                logFileNamer.setAppender("Append");\r
-               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), "./log-Append" + ending + "_1.log");\r
+               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), dir.getCanonicalFile()+"/log-Append" + ending + "_1.log");\r
        }\r
 \r
        @Test\r
        public void testBlankRoot() throws IOException {\r
-               LogFileNamer logFileNamer = new LogFileNamer(".", "");\r
+               LogFileNamer logFileNamer = new LogFileNamer(dir.getCanonicalPath(), "");\r
                assertEquals(logFileNamer, logFileNamer.noPID());\r
 \r
                logFileNamer.setAppender("Append");\r
-               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), "./Append" + ending + "_0.log");\r
+               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), dir.getCanonicalPath()+"/Append" + ending + "_0.log");\r
 \r
                logFileNamer.setAppender("Append");\r
-               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), "./Append" + ending + "_1.log");\r
+               assertEquals(System.getProperty("LOG4J_FILENAME_Append"), dir.getCanonicalPath()+"/Append" + ending + "_1.log");\r
        }\r
 \r
        @After\r