X-Git-Url: https://gerrit.onap.org/r/gitweb?a=blobdiff_plain;f=datarouter-prov%2Fsrc%2Fmain%2Fjava%2Forg%2Fonap%2Fdmaap%2Fdatarouter%2Fprovisioning%2Futils%2FPurgeLogDirTask.java;h=7cb1c02abcba6be125414cb976965cb4e484f9c7;hb=6c78b3e6a0a67c73f931337356a172cc69cee0e8;hp=14d151979a257199ea50f10cc6e7d20f27b4003e;hpb=2db08f1dbee4981422a93d06950226cc749ab08a;p=dmaap%2Fdatarouter.git

diff --git a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/utils/PurgeLogDirTask.java b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/utils/PurgeLogDirTask.java
index 14d15197..7cb1c02a 100644
--- a/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/utils/PurgeLogDirTask.java
+++ b/datarouter-prov/src/main/java/org/onap/dmaap/datarouter/provisioning/utils/PurgeLogDirTask.java
@@ -24,10 +24,14 @@
 
 package org.onap.dmaap.datarouter.provisioning.utils;
 
+import com.att.eelf.configuration.EELFLogger;
+import com.att.eelf.configuration.EELFManager;
 import java.io.File;
+import java.io.IOException;
+import java.nio.file.Files;
+import java.util.Objects;
 import java.util.Properties;
 import java.util.TimerTask;
-import org.apache.log4j.Logger;
 
 /**
  * This class provides a {@link TimerTask} that purges old logfiles (older than the number of days specified by the
@@ -42,15 +46,13 @@ public class PurgeLogDirTask extends TimerTask {
 
     private final String logdir;
     private final long interval;
-    private Logger utilsLogger;
+    private EELFLogger utilsLogger;
 
     public PurgeLogDirTask() {
         Properties p = (new DB()).getProperties();
         logdir = p.getProperty("org.onap.dmaap.datarouter.provserver.accesslog.dir");
         String s = p.getProperty("org.onap.dmaap.datarouter.provserver.logretention", "30");
-
-        this.utilsLogger = Logger.getLogger("org.onap.dmaap.datarouter.provisioning.utils");
-
+        this.utilsLogger = EELFManager.getInstance().getLogger("UtilsLog");
         long n = 30;
         try {
             n = Long.parseLong(s);
@@ -65,15 +67,23 @@ public class PurgeLogDirTask extends TimerTask {
         try {
             File dir = new File(logdir);
             if (dir.exists()) {
-                long exptime = System.currentTimeMillis() - interval;
-                for (File logfile : dir.listFiles()) {
-                    if (logfile.lastModified() < exptime) {
-                        logfile.delete();
-                    }
-                }
+                purgeLogFiles(dir);
             }
         } catch (Exception e) {
-            utilsLogger.error("Exception: " + e.getMessage());
+            utilsLogger.error("Exception: " + e.getMessage(), e);
+        }
+    }
+
+    private void purgeLogFiles(File dir) {
+        long exptime = System.currentTimeMillis() - interval;
+        for (File logfile : Objects.requireNonNull(dir.listFiles())) {
+            if (logfile.lastModified() < exptime) {
+                try {
+                    Files.delete(logfile.toPath());
+                } catch (IOException e) {
+                    utilsLogger.error("Failed to delete file: " + logfile.getPath(), e);
+                }
+            }
         }
     }
 }