Issue #5086 Fix error on addFile

Signed-off-by: Jan Bartel <janb@webtide.com>
jetty · Dec 2, 2020 · e39ec6c · e39ec6c
1 parent 31cc298
commit e39ec6c
Showing 1 changed file with 11 additions and 8 deletions.
diff --git a/jetty-util/src/main/java/org/eclipse/jetty/util/Scanner.java b/jetty-util/src/main/java/org/eclipse/jetty/util/Scanner.java
@@ -349,11 +349,16 @@ public void addFile(Path p)
         if (p == null)
             throw new IllegalStateException("Null path");
 
-        File f = p.toFile();
-        if (!f.exists() || f.isDirectory())
+        if (!Files.exists(p) || Files.isDirectory(p))
             throw new IllegalStateException("Not file or doesn't exist: " + p);
-
-        _scannables.putIfAbsent(p, null);
+        try
+        {
+            _scannables.putIfAbsent(p.toRealPath(), new IncludeExcludeSet<>(PathMatcherSet.class));
+        }
+        catch (IOException e)
+        {
+            throw new IllegalStateException(e);
+        }
     }
 
     /**
@@ -370,15 +375,13 @@ public IncludeExcludeSet<PathMatcher, Path> addDirectory(Path p)
         if (p == null)
             throw new IllegalStateException("Null path");
 
-        File f = p.toFile();
-        if (!f.exists() || !f.isDirectory())
+        if (!Files.exists(p) || !Files.isDirectory(p))
             throw new IllegalStateException("Not directory or doesn't exist: " + p);
 
         try
         {
-            Path real = p.toRealPath();
             IncludeExcludeSet<PathMatcher, Path> includesExcludes = new IncludeExcludeSet<>(PathMatcherSet.class);
-            IncludeExcludeSet<PathMatcher, Path> prev = _scannables.putIfAbsent(real, includesExcludes);
+            IncludeExcludeSet<PathMatcher, Path> prev = _scannables.putIfAbsent(p.toRealPath(), includesExcludes);
             if (prev != null)
                 includesExcludes = prev;
             return includesExcludes;