CB-5609. Refactor after review

dbeaver · Dec 26, 2024 · 97a6704 · 97a6704
1 parent 7ec638d
commit 97a6704
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/...er/bundles/io.cloudbeaver.service.fs/src/io/cloudbeaver/service/fs/impl/WebServiceFS.java b/...er/bundles/io.cloudbeaver.service.fs/src/io/cloudbeaver/service/fs/impl/WebServiceFS.java
@@ -44,7 +44,7 @@
  */
 public class WebServiceFS implements DBWServiceFS {
 
-    public static final String FORBIDDEN_FILENAME_REGEX = "[%#:;№_$]";
+    private static final Pattern FORBIDDEN_FILENAME_PATTERN = Pattern.compile("[%#:;№_$]");
 
     @NotNull
     @Override
@@ -282,12 +282,11 @@ public boolean deleteFile(
         }
     }
 
-    private static void validateFilename(String filename) throws DBWebException {
-        Pattern pattern = Pattern.compile(FORBIDDEN_FILENAME_REGEX);
-        Matcher matcher = pattern.matcher(filename);
+    public void validateFilename(String filename) throws DBWebException {
+        Matcher matcher = FORBIDDEN_FILENAME_PATTERN.matcher(filename);
 
         if (matcher.find()) {
-            throw new DBWebException("File include forbidden symbols: " + filename);
+            throw new DBWebException("File includes forbidden symbols: " + filename);
         }
     }
 }