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=92ab86da43c7f339ce22de45714f90d5a3c7fec4;hb=381d4ebc5e83d5fd5b62fff7e5a6fa6d582149d9;hp=c34e9541ae31bb5dae76e10d24a1cfe8b4ffd9da;hpb=06271b9df78dadfdab95dc2f6f853cdaa3148d9d;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 c34e9541..92ab86da 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
@@ -27,6 +27,8 @@ 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;
@@ -46,18 +48,21 @@ public class PurgeLogDirTask extends TimerTask {
     private final long interval;
     private EELFLogger utilsLogger;
 
+    /**
+     * PurgeLogDirTask constructor.
+     */
     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");
+        Properties prop = (new DB()).getProperties();
+        logdir = prop.getProperty("org.onap.dmaap.datarouter.provserver.accesslog.dir");
+        String str = prop.getProperty("org.onap.dmaap.datarouter.provserver.logretention", "30");
         this.utilsLogger = EELFManager.getInstance().getLogger("UtilsLog");
-        long n = 30;
+        long retention = 30;
         try {
-            n = Long.parseLong(s);
+            retention = Long.parseLong(str);
         } catch (NumberFormatException e) {
             // ignore
         }
-        interval = n * ONEDAY;
+        interval = retention * ONEDAY;
     }
 
     @Override
@@ -76,7 +81,11 @@ public class PurgeLogDirTask extends TimerTask {
         long exptime = System.currentTimeMillis() - interval;
         for (File logfile : Objects.requireNonNull(dir.listFiles())) {
             if (logfile.lastModified() < exptime) {
-                logfile.delete();
+                try {
+                    Files.delete(logfile.toPath());
+                } catch (IOException e) {
+                    utilsLogger.error("Failed to delete file: " + logfile.getPath(), e);
+                }
             }
         }
     }