updated CacheFilter to clean dead comments and import Paths/Path; improved StaticFileHandler path traversal logic for better security and simpler validation

Author: Boppler12

src/main/java/org/example/CacheFilter.java

@@ -2,6 +2,8 @@
 package org.example;
 
 import java.io.IOException;
+import java.nio.file.Path;
+import java.nio.file.Paths;
 import java.util.Comparator;
 import java.util.concurrent.ConcurrentHashMap;
 import java.util.concurrent.atomic.AtomicLong;
@@ -68,6 +70,8 @@ public byte[] getOrFetch(String uri, FileProvider provider) throws IOException {
                 LOGGER.log(Level.FINE, "✓ Cache hit for: " + uri + " (from concurrent fetch)");
                 return entry.data;
             }
+            
+            
 
             // Fetch och cachelagra
             LOGGER.log(Level.FINE, "✗ Cache miss for: " + uri);
@@ -81,9 +85,6 @@ public byte[] getOrFetch(String uri, FileProvider provider) throws IOException {
         }
     }
 
-    /**
-     * Lägg till i cache med eviction om nödvändigt (MÅSTE VARA UNDER LOCK)
-     */
     private void addToCacheUnsafe(String uri, byte[] data) {
         // Guard mot oversized entries som kan blockera eviction
         if (data.length > MAX_CACHE_BYTES) {

src/main/java/org/example/StaticFileHandler.java

@@ -96,17 +96,22 @@ private String sanitizeUri(String uri) {
     }
 
     /**
-     * Checks if the requested path attempts to traverse outside the web root.
-     * Uses path normalization after decoding to catch traversal attempts.
+    /**
+     * Kontrollerar om den begärda sökvägen försöker traversera utanför webroten.
+     * Använder sökvägsnormalisering efter avkodning för att fånga traversalförsök.
      */
     private boolean isPathTraversal(String uri) {
         try {
-            Path webRootPath = Paths.get(webRoot).toRealPath();
+            // Använd absolutsökväg + normalisera istället för toRealPath() för att undvika
+            // krav på att katalogen existerar och för att hantera symboliska länkar säkert
+            Path webRootPath = Paths.get(webRoot).toAbsolutePath().normalize();
             Path requestedPath = webRootPath.resolve(uri).normalize();
-            
+
+            // Returnera true om den begärda sökvägen inte ligger under webroten
             return !requestedPath.startsWith(webRootPath);
-        } catch (IOException e) {
-            LOGGER.log(Level.WARNING, "Path traversal check failed for: " + uri, e);
+        } catch (Exception e) {
+            // Om något går fel under sökvägsvalideringen, tillåt inte åtkomst (säker utgång)
+            LOGGER.log(Level.WARNING, "Sökvägstraversalkontroll misslyckades för: " + uri, e);
             return true;
         }
     }