epugh commented on code in PR #4672:
URL: https://github.com/apache/solr/pull/4672#discussion_r4116399778


##########
solr/core/src/java/org/apache/solr/core/ConfigSetService.java:
##########
@@ -75,6 +81,65 @@ public static boolean isFileForbiddenInConfigSets(String 
filePath) {
     return lastDot >= 0 && 
USE_FORBIDDEN_FILE_TYPES.contains(filePath.substring(lastDot + 1));
   }
 
+  /**
+   * Zips the contents of {@code rootPath} into an in-memory archive. Hidden 
files and directories
+   * (as determined by {@link Files#isHidden}) are skipped, directory entries 
are written for
+   * non-empty subdirectories, and zip entry names are normalized to use 
{@code /} separators
+   * regardless of platform.
+   *
+   * @param rootPath the directory to zip
+   * @param validateFileTypes if true, a file with a forbidden extension (see 
{@link
+   *     #isFileForbiddenInConfigSets}) causes an {@link IOException} instead 
of being silently

Review Comment:
   there are a couple of pattenrs, but ended up with `IllegalArgumentException` 
as the closes to what a CLI needs.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to