backend: (GitLab) URL Reader uses content-disposition header for filename

packages/backend-common/src/reading/GitlabUrlReader.test.ts

@@ -176,6 +176,10 @@ describe('GitlabUrlReader', () => {
             res(
               ctx.status(200),
               ctx.set('Content-Type', 'application/zip'),
+              ctx.set(
+                'content-disposition',
+                'attachment; filename="mock-main-sha123abc.zip"',
+              ),
               ctx.body(archiveBuffer),
             ),
         ),
@@ -225,6 +229,10 @@ describe('GitlabUrlReader', () => {
             res(
               ctx.status(200),
               ctx.set('Content-Type', 'application/zip'),
+              ctx.set(
+                'content-disposition',
+                'attachment; filename="mock-main-sha123abc.zip"',
+              ),
               ctx.body(archiveBuffer),
             ),
         ),
@@ -254,6 +262,10 @@ describe('GitlabUrlReader', () => {
             res(
               ctx.status(200),
               ctx.set('Content-Type', 'application/zip'),
+              ctx.set(
+                'content-disposition',
+                'attachment; filename="mock-main-sha123abc.zip"',
+              ),
               ctx.body(archiveBuffer),
             ),
         ),

packages/backend-common/src/reading/GitlabUrlReader.ts

@@ -140,9 +140,29 @@ export class GitlabUrlReader implements UrlReader {
       throw new Error(message);
     }
 
-    const path = filepath
-      ? `${repoName}-${branch}-${commitSha}/${filepath}/`
-      : '';
+    // Get the filename of archive from the header of the response
+    const contentDispositionHeader = archiveGitLabResponse.headers.get(
+      'content-disposition',
+    ) as string;
+    if (!contentDispositionHeader) {
+      throw new Error(
+        `Failed to read tree from ${url}. ` +
+          'GitLab API response for downloading archive does not contain content-disposition header ',
+      );
+    }
+    const fileNameRegEx = new RegExp(
+      /^attachment; filename="(?<fileName>.*).zip"$/,
+    );
+    const archiveFileName = contentDispositionHeader.match(fileNameRegEx)
+      ?.groups?.fileName;
+    if (!archiveFileName) {
+      throw new Error(
+        `Failed to read tree from ${url}. GitLab API response for downloading archive has an unexpected ` +
+          `format of content-disposition header ${contentDispositionHeader} `,
+      );
+    }
+
+    const path = filepath ? `${archiveFileName}/${filepath}/` : '';
 
     return await this.treeResponseFactory.fromZipArchive({
       stream: (archiveGitLabResponse.body as unknown) as Readable,