Apache
/
lucene
mirror of https://github.com/apache/lucene.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

SOLR-7125: Don't upload dotfiles 

git-svn-id: https://svn.apache.org/repos/asf/lucene/dev/trunk@1661506 13f79535-47bb-0310-9956-ffa450edef68
This commit is contained in:
Alan Woodward 2015-02-22 18:22:40 +00:00
parent bf83ae2a44
commit 1fdf68d482
2 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
solr/solrj/src/java/org/apache/solr/common/cloud/ZkConfigManager.java
View File

@ -58,6 +58,9 @@ public class ZkConfigManager {
Files.walkFileTree(rootPath, new SimpleFileVisitor<Path>(){
@Override
public FileVisitResult visitFile(Path file, BasicFileAttributes attrs) throws IOException {
String filename = file.getFileName().toString();
if (filename.startsWith("."))
return FileVisitResult.CONTINUE;
String zkNode = zkPath + "/" + rootPath.relativize(file).toString();
try {
zkClient.makePath(zkNode, file.toFile(), false, true);
@ -67,6 +70,11 @@ public class ZkConfigManager {
}
return FileVisitResult.CONTINUE;
}
@Override
public FileVisitResult preVisitDirectory(Path dir, BasicFileAttributes attrs) throws IOException {
return (dir.getFileName().toString().startsWith(".")) ? FileVisitResult.SKIP_SUBTREE : FileVisitResult.CONTINUE;
}
});
}

6
solr/solrj/src/test/org/apache/solr/common/cloud/TestZkConfigManager.java
View File

@ -71,6 +71,9 @@ public class TestZkConfigManager extends SolrTestCaseJ4 {
Files.createFile(tempConfig.resolve("file2"));
Files.createDirectory(tempConfig.resolve("subdir"));
Files.createFile(tempConfig.resolve("subdir").resolve("file3"));
Files.createFile(tempConfig.resolve(".ignored"));
Files.createDirectory(tempConfig.resolve(".ignoreddir"));
Files.createFile(tempConfig.resolve(".ignoreddir").resolve("ignored"));
configManager.uploadConfigDir(tempConfig, "testconfig");
@ -86,6 +89,9 @@ public class TestZkConfigManager extends SolrTestCaseJ4 {
assertTrue(Files.exists(downloadPath.resolve("file2")));
assertTrue(Files.isDirectory(downloadPath.resolve("subdir")));
assertTrue(Files.exists(downloadPath.resolve("subdir/file3")));
// dotfiles should be ignored
assertFalse(Files.exists(downloadPath.resolve(".ignored")));
assertFalse(Files.exists(downloadPath.resolve(".ignoreddir/ignored")));
byte[] checkdata = Files.readAllBytes(downloadPath.resolve("file1"));
assertArrayEquals(testdata, checkdata);