epugh commented on code in PR #4672:
URL: https://github.com/apache/solr/pull/4672#discussion_r4116373448
##########
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:
fair question... I'm looking to see if we have any patterns elsewhere in
the code base for this situation.
--
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]
