#982 unable to find path for getArchive api

Added "path" parameter to methods

Fixes #982

Author: DoubleDPro

src/main/java/org/gitlab4j/api/RepositoryApi.java

@@ -443,14 +443,35 @@ public InputStream getRawBlobContent(Object projectIdOrPath, String sha) throws
      * @return an input stream that can be used to save as a file
      * or to read the content of the archive
      * @throws GitLabApiException if any exception occurs
+     * @deprecated Use {@link #getRepositoryArchiveByPathAndSha(Object, String, String)}
      */
+    @Deprecated
     public InputStream getRepositoryArchive(Object projectIdOrPath, String sha) throws GitLabApiException {
         Form formData = new GitLabApiForm().withParam("sha", sha);
         Response response = getWithAccepts(Response.Status.OK, formData.asMap(), MediaType.WILDCARD,
                 "projects", getProjectIdOrPath(projectIdOrPath), "repository", "archive");
         return (response.readEntity(InputStream.class));
     }
 
+    /**
+     * Get an archive of the complete repository by SHA (optional).
+     *
+     * <pre><code>GitLab Endpoint: GET /projects/:id/repository/archive</code></pre>
+     *
+     * @param projectIdOrPath the project in the form of an Long(ID), String(path), or Project instance
+     * @param path The subpath of the repository to download
+     * @param sha the SHA of the archive to get
+     * @return an input stream that can be used to save as a file
+     * or to read the content of the archive
+     * @throws GitLabApiException if any exception occurs
+     */
+    public InputStream getRepositoryArchiveByPathAndSha(Object projectIdOrPath, String path, String sha) throws GitLabApiException {
+        Form formData = new GitLabApiForm().withParam("sha", sha).withParam("path", path);
+        Response response = getWithAccepts(Response.Status.OK, formData.asMap(), MediaType.WILDCARD,
+            "projects", getProjectIdOrPath(projectIdOrPath), "repository", "archive");
+        return (response.readEntity(InputStream.class));
+    }
+
     /**
      * Get an archive of the complete repository by SHA (optional).
      *
@@ -461,12 +482,31 @@ public InputStream getRepositoryArchive(Object projectIdOrPath, String sha) thro
      * @param format The archive format, defaults to "tar.gz" if null
      * @return an input stream that can be used to save as a file or to read the content of the archive
      * @throws GitLabApiException if format is not a valid archive format or any exception occurs
+     * @deprecated Use {@link #getRepositoryArchiveByPathAndSha(Object, String, String, String)}
      */
+    @Deprecated
     public InputStream getRepositoryArchive(Object projectIdOrPath, String sha, String format) throws GitLabApiException {
         ArchiveFormat archiveFormat = ArchiveFormat.forValue(format);
         return (getRepositoryArchive(projectIdOrPath, sha, archiveFormat));
     }
 
+    /**
+     * Get an archive of the complete repository by SHA (optional).
+     *
+     * <pre><code>GitLab Endpoint: GET /projects/:id/repository/archive</code></pre>
+     *
+     * @param projectIdOrPath the project in the form of an Long(ID), String(path), or Project instance
+     * @param path The subpath of the repository to download
+     * @param sha the SHA of the archive to get
+     * @param format The archive format, defaults to "tar.gz" if null
+     * @return an input stream that can be used to save as a file or to read the content of the archive
+     * @throws GitLabApiException if format is not a valid archive format or any exception occurs
+     */
+    public InputStream getRepositoryArchiveByPathAndSha(Object projectIdOrPath, String path, String sha, String format) throws GitLabApiException {
+        ArchiveFormat archiveFormat = ArchiveFormat.forValue(format);
+        return (getRepositoryArchiveByPathAndSha(projectIdOrPath, path, sha, archiveFormat));
+    }
+
     /**
      * Get an archive of the complete repository by SHA (optional).
      *
@@ -477,7 +517,9 @@ public InputStream getRepositoryArchive(Object projectIdOrPath, String sha, Stri
      * @param format The archive format, defaults to TAR_GZ if null
      * @return an input stream that can be used to save as a file or to read the content of the archive
      * @throws GitLabApiException if any exception occurs
+     * @deprecated User {@link #getRepositoryArchiveByPathAndSha(Object, String, String, ArchiveFormat)}
      */
+    @Deprecated
     public InputStream getRepositoryArchive(Object projectIdOrPath, String sha, ArchiveFormat format) throws GitLabApiException {
 
         if (format == null) {
@@ -497,6 +539,37 @@ public InputStream getRepositoryArchive(Object projectIdOrPath, String sha, Arch
         return (response.readEntity(InputStream.class));
     }
 
+    /**
+     * Get an archive of the complete repository by SHA (optional).
+     *
+     * <pre><code>GitLab Endpoint: GET /projects/:id/repository/archive</code></pre>
+     *
+     * @param projectIdOrPath the project in the form of an Long(ID), String(path), or Project instance
+     * @param path The subpath of the repository to download
+     * @param sha the SHA of the archive to get
+     * @param format The archive format, defaults to TAR_GZ if null
+     * @return an input stream that can be used to save as a file or to read the content of the archive
+     * @throws GitLabApiException if any exception occurs
+     */
+    public InputStream getRepositoryArchiveByPathAndSha(Object projectIdOrPath, String path, String sha, ArchiveFormat format) throws GitLabApiException {
+
+        if (format == null) {
+            format = ArchiveFormat.TAR_GZ;
+        }
+
+        /*
+         * Gitlab-ce has a bug when you try to download file archives with format by using "&format=zip(or tar... etc.)",
+         * there is a solution to request .../archive.:format instead of .../archive?format=:format.
+         *
+         * Issue:  https://gitlab.com/gitlab-org/gitlab-ce/issues/45992
+         *         https://gitlab.com/gitlab-com/support-forum/issues/3067
+         */
+        Form formData = new GitLabApiForm().withParam("path", path).withParam("sha", sha);
+        Response response = getWithAccepts(Response.Status.OK, formData.asMap(), MediaType.WILDCARD,
+            "projects", getProjectIdOrPath(projectIdOrPath), "repository", "archive" + "." + format.toString());
+        return (response.readEntity(InputStream.class));
+    }
+
     /**
      * Get an archive of the complete repository by SHA (optional) and saves to the specified directory.
      * If the archive already exists in the directory it will be overwritten.
@@ -508,7 +581,9 @@ public InputStream getRepositoryArchive(Object projectIdOrPath, String sha, Arch
      * @param directory the File instance of the directory to save the archive to, if null will use "java.io.tmpdir"
      * @return a File instance pointing to the downloaded instance
      * @throws GitLabApiException if any exception occurs
+     * @deprecated Use {@link #getRepositoryArchiveByPathAndSha(Object, String, String, File)}
      */
+    @Deprecated
     public File getRepositoryArchive(Object projectIdOrPath, String sha, File directory) throws GitLabApiException {
 
         Form formData = new GitLabApiForm().withParam("sha", sha);
@@ -532,6 +607,42 @@ public File getRepositoryArchive(Object projectIdOrPath, String sha, File direct
         }
     }
 
+    /**
+     * Get an archive of the complete repository by SHA (optional) and saves to the specified directory.
+     * If the archive already exists in the directory it will be overwritten.
+     *
+     * <pre><code>GitLab Endpoint: GET /projects/:id/repository/archive</code></pre>
+     *
+     * @param projectIdOrPath the project in the form of an Long(ID), String(path), or Project instance
+     * @param path The subpath of the repository to download
+     * @param sha the SHA of the archive to get
+     * @param directory the File instance of the directory to save the archive to, if null will use "java.io.tmpdir"
+     * @return a File instance pointing to the downloaded instance
+     * @throws GitLabApiException if any exception occurs
+     */
+    public File getRepositoryArchiveByPathAndSha(Object projectIdOrPath, String path, String sha, File directory) throws GitLabApiException {
+
+        Form formData = new GitLabApiForm().withParam("path", path).withParam("sha", sha);
+        Response response = getWithAccepts(Response.Status.OK, formData.asMap(), MediaType.WILDCARD,
+            "projects", getProjectIdOrPath(projectIdOrPath), "repository", "archive");
+
+        try {
+
+            if (directory == null)
+                directory = new File(System.getProperty("java.io.tmpdir"));
+
+            String filename = FileUtils.getFilenameFromContentDisposition(response);
+            File file = new File(directory, filename);
+
+            InputStream in = response.readEntity(InputStream.class);
+            Files.copy(in, file.toPath(), StandardCopyOption.REPLACE_EXISTING);
+            return (file);
+
+        } catch (IOException ioe) {
+            throw new GitLabApiException(ioe);
+        }
+    }
+
     /**
      * Get an archive of the complete repository by SHA (optional) and saves to the specified directory.
      * If the archive already exists in the directory it will be overwritten.
@@ -544,12 +655,33 @@ public File getRepositoryArchive(Object projectIdOrPath, String sha, File direct
      * @param format The archive format, defaults to "tar.gz" if null
      * @return a File instance pointing to the downloaded instance
      * @throws GitLabApiException if format is not a valid archive format or any exception occurs
+     * @deprecated Use {@link #getRepositoryArchiveByPathAndSha(Object, String, String, File, String)}
      */
+    @Deprecated
     public File getRepositoryArchive(Object projectIdOrPath, String sha, File directory, String format) throws GitLabApiException {
         ArchiveFormat archiveFormat = ArchiveFormat.forValue(format);
         return (getRepositoryArchive(projectIdOrPath, sha, directory, archiveFormat));
     }
 
+    /**
+     * Get an archive of the complete repository by SHA (optional) and saves to the specified directory.
+     * If the archive already exists in the directory it will be overwritten.
+     *
+     * <pre><code>GitLab Endpoint: GET /projects/:id/repository/archive</code></pre>
+     *
+     * @param projectIdOrPath the project in the form of an Long(ID), String(path), or Project instance
+     * @param path The subpath of the repository to download
+     * @param sha the SHA of the archive to get
+     * @param directory the File instance of the directory to save the archive to, if null will use "java.io.tmpdir"
+     * @param format The archive format, defaults to "tar.gz" if null
+     * @return a File instance pointing to the downloaded instance
+     * @throws GitLabApiException if format is not a valid archive format or any exception occurs
+     */
+    public File getRepositoryArchiveByPathAndSha(Object projectIdOrPath, String path, String sha, File directory, String format) throws GitLabApiException {
+        ArchiveFormat archiveFormat = ArchiveFormat.forValue(format);
+        return (getRepositoryArchiveByPathAndSha(projectIdOrPath, path, sha, directory, archiveFormat));
+    }
+
     /**
      * Get an archive of the complete repository by SHA (optional) and saves to the specified directory.
      * If the archive already exists in the directory it will be overwritten.
@@ -562,7 +694,9 @@ public File getRepositoryArchive(Object projectIdOrPath, String sha, File direct
      * @param format The archive format, defaults to TAR_GZ if null
      * @return a File instance pointing to the downloaded instance
      * @throws GitLabApiException if any exception occurs
+     * @deprecated Use {@link #getRepositoryArchiveByPathAndSha(Object, String, String, File, ArchiveFormat)}
      */
+    @Deprecated
     public File getRepositoryArchive(Object projectIdOrPath, String sha, File directory, ArchiveFormat format) throws GitLabApiException {
 
         if (format == null) {
@@ -597,6 +731,54 @@ public File getRepositoryArchive(Object projectIdOrPath, String sha, File direct
         }
     }
 
+    /**
+     * Get an archive of the complete repository by SHA (optional) and saves to the specified directory.
+     * If the archive already exists in the directory it will be overwritten.
+     *
+     * <pre><code>GitLab Endpoint: GET /projects/:id/repository/archive</code></pre>
+     *
+     * @param projectIdOrPath the project in the form of an Long(ID), String(path), or Project instance
+     * @param path The subpath of the repository to download
+     * @param sha the SHA of the archive to get
+     * @param directory the File instance of the directory to save the archive to, if null will use "java.io.tmpdir"
+     * @param format The archive format, defaults to TAR_GZ if null
+     * @return a File instance pointing to the downloaded instance
+     * @throws GitLabApiException if any exception occurs
+     */
+    public File getRepositoryArchiveByPathAndSha(Object projectIdOrPath, String path, String sha, File directory, ArchiveFormat format) throws GitLabApiException {
+
+        if (format == null) {
+            format = ArchiveFormat.TAR_GZ;
+        }
+
+        /*
+         * Gitlab-ce has a bug when you try to download file archives with format by using "&format=zip(or tar... etc.)",
+         * there is a solution to request .../archive.:format instead of .../archive?format=:format.
+         *
+         * Issue:  https://gitlab.com/gitlab-org/gitlab-ce/issues/45992
+         *         https://gitlab.com/gitlab-com/support-forum/issues/3067
+         */
+        Form formData = new GitLabApiForm().withParam("path", path).withParam("sha", sha);
+        Response response = getWithAccepts(Response.Status.OK, formData.asMap(), MediaType.WILDCARD,
+            "projects", getProjectIdOrPath(projectIdOrPath), "repository", "archive" + "." + format.toString());
+
+        try {
+
+            if (directory == null)
+                directory = new File(System.getProperty("java.io.tmpdir"));
+
+            String filename = FileUtils.getFilenameFromContentDisposition(response);
+            File file = new File(directory, filename);
+
+            InputStream in = response.readEntity(InputStream.class);
+            Files.copy(in, file.toPath(), StandardCopyOption.REPLACE_EXISTING);
+            return (file);
+
+        } catch (IOException ioe) {
+            throw new GitLabApiException(ioe);
+        }
+    }
+
     /**
      * Compare branches, tags or commits. This can be accessed without authentication
      * if the repository is publicly accessible.

src/test/java/org/gitlab4j/api/TestRepositoryApi.java

@@ -15,6 +15,7 @@
 import java.util.Arrays;
 import java.util.List;
 
+import org.gitlab4j.api.Constants.ArchiveFormat;
 import org.gitlab4j.api.models.Branch;
 import org.gitlab4j.api.models.Commit;
 import org.gitlab4j.api.models.CompareResults;
@@ -142,6 +143,39 @@ public void testRepositoryArchiveViaInputStream() throws GitLabApiException, IOE
         Files.delete(target);
     }
 
+    @Test
+    public void testRepositoryArchiveByPathAndSha() throws IOException, GitLabApiException {
+
+        Project project = gitLabApi.getProjectApi().getProject(TEST_NAMESPACE, TEST_PROJECT_NAME);
+        assertNotNull(project);
+
+        InputStream in = gitLabApi.getRepositoryApi()
+            .getRepositoryArchiveByPathAndSha(project.getId(), project.getPath(), "master");
+
+        Path target = Files.createTempFile(TEST_PROJECT_NAME + "-", ".tar.gz");
+        Files.copy(in, target, StandardCopyOption.REPLACE_EXISTING);
+
+        assertTrue(target.toFile().length() > 0);
+        Files.delete(target);
+
+    }
+
+    @Test
+    public void testRepositoryArchiveByPathAndShaAndFormat() throws GitLabApiException, IOException {
+
+        Project project = gitLabApi.getProjectApi().getProject(TEST_NAMESPACE, TEST_PROJECT_NAME);
+        assertNotNull(project);
+
+        InputStream in = gitLabApi.getRepositoryApi()
+            .getRepositoryArchiveByPathAndSha(project.getId(), project.getPath(), "master", ArchiveFormat.TAR_GZ.toString());
+
+        Path target = Files.createTempFile(TEST_PROJECT_NAME + "-", ".tar.gz");
+        Files.copy(in, target, StandardCopyOption.REPLACE_EXISTING);
+
+        assertTrue(target.toFile().length() > 0);
+        Files.delete(target);
+    }
+
     @Test
     public void testRepositoryArchiveViaFile() throws GitLabApiException, IOException {