Fix signed int causing issues calculating expiry time for images

bundabrg · Jul 21, 2020 · aad3c23 · aad3c23
1 parent 902b2e0
commit aad3c23
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/connector/src/main/java/org/geysermc/connector/utils/SkinProvider.java b/connector/src/main/java/org/geysermc/connector/utils/SkinProvider.java
@@ -113,16 +113,19 @@ public class SkinProvider {
                 }
 
                 int count = 0;
+                final long expireTime = ((long)GeyserConnector.getInstance().getConfig().getCacheImages()) * ((long)1000 * 60 * 60 * 24);
                 for (File imageFile : Objects.requireNonNull(cacheFolder.listFiles())) {
-                    if (imageFile.lastModified() < System.currentTimeMillis() - (GeyserConnector.getInstance().getConfig().getCacheImages() * 1000 * 60 * 60 * 24)) {
+                    if (imageFile.lastModified() < System.currentTimeMillis() - expireTime) {
                         //noinspection ResultOfMethodCallIgnored
                         imageFile.delete();
                         count++;
                     }
                 }
 
-                GeyserConnector.getInstance().getLogger().debug(String.format("Removed %d cached image files as they have expired", count));
-            }, 1, 1440, TimeUnit.MINUTES);
+                if (count > 0) {
+                    GeyserConnector.getInstance().getLogger().debug(String.format("Removed %d cached image files as they have expired", count));
+                }
+            }, 10, 1440, TimeUnit.MINUTES);
         }
     }