Further limit allowed characters in file path (#686)

* Further limit allowed characters in file path

* Apply suggestions from code review

Co-authored-by: Zbynek Konecny <[email protected]>

---------

Co-authored-by: Daniel Beck <[email protected]>
Co-authored-by: Zbynek Konecny <[email protected]>

Author: daniel-beck

src/main/java/io/jenkins/update_center/ArtifactoryRepositoryImpl.java

@@ -79,15 +79,20 @@ protected Set<ArtifactCoordinates> listAllJenkinsWars(String groupId) throws IOE
     }
 
     private static boolean containsIllegalChars(String test) {
-        return !test.chars().allMatch(c -> c >= 0x2B && c < 0x7B);
+        return !test.chars().allMatch(c ->
+                c >= '0' && c <= '9'
+                || c >= 'A' && c <= 'Z'
+                || c >= 'a' && c <= 'z'
+                || c == '+' || c == '-' || c == '.' || c == '/' || c == '_'
+        );
     }
 
     private static ArtifactCoordinates toGav(JsonFile f) {
         String fileName = f.name;
         String path = f.path;
 
         if (containsIllegalChars(fileName) || containsIllegalChars(path)) {
-            LOGGER.log(Level.INFO, "Not only printable ascii: " + f.path + " / " + f.name);
+            LOGGER.log(Level.INFO, "Characters outside allowed set: " + f.path + " / " + f.name);
             return null;
         }