From aad3c2357158b4f8264c7ba4ed5676bf3a6297ef Mon Sep 17 00:00:00 2001 From: bundabrg Date: Tue, 21 Jul 2020 13:04:50 +0800 Subject: [PATCH] Fix signed int causing issues calculating expiry time for images --- .../java/org/geysermc/connector/utils/SkinProvider.java | 9 ++++++--- 1 file changed, 6 insertions(+), 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 index 8d26065a13c..26b7b02c9e2 100644 --- 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); } }