Do not resolve symbolic links to repositories.

When symbolic links under the base repository folder point to repositories
outside the base repository folder, the forming of relative repository
names failed and resulted in NullPointerExceptions.

Create the relative path by not following symbolic links, i.e. the link
name is taken as is and not resolved to the external path.

This also changes the whole `exactPath` method to work on Paths, instead
of Files.

Fixes #891 and fixes #837.

1 files changed, 117 insertions, 0 deletions

diff --git a/src/test/java/com/gitblit/tests/JGitUtilsTest.java b/src/test/java/com/gitblit/tests/JGitUtilsTest.java
index b133a8c8..5f083555 100644
--- a/src/test/java/com/gitblit/tests/JGitUtilsTest.java
+++ b/src/test/java/com/gitblit/tests/JGitUtilsTest.java
@@ -17,6 +17,10 @@ package com.gitblit.tests;
 

 import java.io.File;

 import java.io.FileOutputStream;

+import java.io.IOException;

+import java.nio.file.Files;

+import java.nio.file.Path;

+import java.nio.file.Paths;

 import java.text.SimpleDateFormat;

 import java.util.Arrays;

 import java.util.Date;

@@ -51,6 +55,8 @@ import com.gitblit.utils.JGitUtils;
 import com.gitblit.utils.JnaUtils;

 import com.gitblit.utils.StringUtils;

 

+import static org.junit.Assume.assumeTrue;

+

 public class JGitUtilsTest extends GitblitUnitTest {

 

 	@Test

@@ -75,6 +81,117 @@ public class JGitUtilsTest extends GitblitUnitTest {
 	}

 

 	@Test

+	public void testFindRepositoriesSymLinked() {

+

+		String reposdirName = "gitrepos";

+		String repositoryName = "test-linked.git";

+		File extrepodir = new File(GitBlitSuite.REPOSITORIES.getParent(), reposdirName);

+		Path symlink = null;

+		Path alink = null;

+		try {

+			Path link = Paths.get( GitBlitSuite.REPOSITORIES.toString(),  "test-rln.git");

+			Path target = Paths.get("../" + reposdirName,repositoryName);

+			symlink = Files.createSymbolicLink(link, target);

+

+			link = Paths.get( GitBlitSuite.REPOSITORIES.toString(),  "test-ln.git");

+			target = Paths.get(extrepodir.getCanonicalPath(), repositoryName);

+			alink = Files.createSymbolicLink(link, target);

+		}

+		catch (UnsupportedOperationException e) {

+			assumeTrue("No symbolic links supported.", false);

+		}

+		catch (IOException ioe) {

+			try {

+				if (symlink != null) Files.delete(symlink);

+				if (alink != null) Files.delete(alink);

+			}

+			catch (IOException ignored) {}

+			fail(ioe.getMessage());

+		}

+

+

+		Path extDir = null;

+		Repository repository = null;

+

+		String testDirName = "test-linked";

+		String testTestDirName = "test-linked/test";

+		Path testDir = Paths.get(GitBlitSuite.REPOSITORIES.toString(),testDirName);

+		Path testTestDir = Paths.get(GitBlitSuite.REPOSITORIES.toString(),testTestDirName);

+		Path linkTestRepo = null;

+		Path linkTestTestRepo = null;

+		Path linkExtDir = null;

+		try {

+			List<String> list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null);

+			int preSize = list.size();

+

+			// Create test repo. This will make the link targets exist, so that the number of repos increases by two.

+			extDir = Files.createDirectory(extrepodir.toPath());

+			repository = JGitUtils.createRepository(extrepodir, repositoryName);

+

+			list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null);

+			assertEquals("No linked repositories found in " + GitBlitSuite.REPOSITORIES, 2, (list.size() - preSize));

+

+			list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, Arrays.asList(".*ln\\.git"));

+			assertEquals("Filtering out linked repos failed.", preSize, list.size());

+

+			// Create subdirectories and place links into them

+			Files.createDirectories(testTestDir);

+

+			Path target = Paths.get(extrepodir.getCanonicalPath(), repositoryName);

+			Path link = Paths.get(testDir.toString(), "test-ln-one.git");

+			linkTestRepo = Files.createSymbolicLink(link, target);

+			link = Paths.get(testTestDir.toString(), "test-ln-two.git");

+			linkTestTestRepo = Files.createSymbolicLink(link, target);

+

+			list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null);

+			assertEquals("No linked repositories found in subdirectories of " + GitBlitSuite.REPOSITORIES, 4, (list.size() - preSize));

+			assertTrue("Did not find linked repo test-ln-one.git", list.contains(testDirName + "/test-ln-one.git"));

+			assertTrue("Did not find linked repo test-ln-two.git", list.contains(testTestDirName + "/test-ln-two.git"));

+			list = JGitUtils.getRepositoryList(new File(testDir.toString()), true, true, -1, null);

+			assertEquals("No linked repositories found in subdirectories of " + testDir, 2, list.size());

+			assertTrue("Did not find linked repo test-ln-one.git", list.contains("test-ln-one.git"));

+			assertTrue("Did not find linked repo test-ln-two.git", list.contains("test/test-ln-two.git"));

+

+

+			// Create link to external directory with repos

+			target = Paths.get(extrepodir.getCanonicalPath());

+			link = Paths.get(testDir.toString(), "test-linked");

+			linkExtDir = Files.createSymbolicLink(link, target);

+

+			list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, true, true, -1, null);

+			assertEquals("No repositories found in linked subdirectories of " + GitBlitSuite.REPOSITORIES, 5, (list.size() - preSize));

+			assertTrue("Did not find repo in linked subfolder.", list.contains(testDirName + "/test-linked/" + repositoryName));

+

+			list = JGitUtils.getRepositoryList(new File(testDir.toString()), true, true, -1, null);

+			assertEquals("No repositories found in linked subdirectories of " + testDir, 3, list.size());

+			assertTrue("Did not find repo in linked subfolder.", list.contains("test-linked/" + repositoryName));

+

+		} catch (IOException e) {

+			fail(e.toString());

+		} finally {

+			try {

+				if (repository != null) {

+					repository.close();

+					RepositoryCache.close(repository);

+					FileUtils.delete(repository.getDirectory(), FileUtils.RECURSIVE);

+				}

+				if (extDir != null) Files.delete(extDir);

+				if (symlink != null) Files.delete(symlink);

+				if (alink != null) Files.delete(alink);

+

+				if (linkExtDir != null) Files.deleteIfExists(linkExtDir);

+				if (linkTestTestRepo != null) Files.deleteIfExists(linkTestTestRepo);

+				if (linkTestRepo != null) Files.deleteIfExists(linkTestRepo);

+

+				Files.deleteIfExists(testTestDir);

+				Files.deleteIfExists(testDir);

+			}

+			catch (IOException ignored) {}

+		}

+

+	}

+

+	@Test

 	public void testFindExclusions() {

 		List<String> list = JGitUtils.getRepositoryList(GitBlitSuite.REPOSITORIES, false, true, -1, null);

 		assertTrue("Missing jgit repository?!", list.contains("test/jgit.git"));