From 47a6a04b2b03e7f9e11d141415c22c3deede0221 Mon Sep 17 00:00:00 2001
From: "Cornelius A. Ludmann" <cornelius@gitpod.io>
Date: Tue, 24 Aug 2021 16:07:09 +0000
Subject: [PATCH] [gitlab] Accept '#' sign in branches / context URLs

---
 .../src/gitlab/gitlab-context-parser.spec.ts  | 60 +++++++++++++++++++
 .../src/gitlab/gitlab-context-parser.ts       |  4 +-
 2 files changed, 63 insertions(+), 1 deletion(-)

diff --git a/components/server/src/gitlab/gitlab-context-parser.spec.ts b/components/server/src/gitlab/gitlab-context-parser.spec.ts
index 2ae5854967d89e..c5a4211fbb3f35 100644
--- a/components/server/src/gitlab/gitlab-context-parser.spec.ts
+++ b/components/server/src/gitlab/gitlab-context-parser.spec.ts
@@ -528,6 +528,66 @@ class TestGitlabContextParser {
         })
     }
 
+    @test public async testTreeContextBranchWithHashSign01() {
+        const result = await this.parser.handle({}, this.user, 'https://gitlab.com/gp-test-group/gp-test-project/-/tree/feature/%23123-issue');
+        expect(result).to.deep.include({
+            "ref": "feature/#123-issue",
+            "refType": "branch",
+            "path": "",
+            "revision": "8b389233c0a3a55a5cd75f89d2c96761420bf2c8",
+            "isFile": false,
+            "repository": {
+                "host": "gitlab.com",
+                "owner": "gp-test-group",
+                "name": "gp-test-project",
+                "cloneUrl": "https://gitlab.com/gp-test-group/gp-test-project.git",
+                "defaultBranch": "master",
+                "private": false
+            },
+            "title": "gp-test-group/gp-test-project - feature/#123-issue"
+        })
+    }
+
+    @test public async testTreeContextBranchWithHashSign02() {
+        const result = await this.parser.handle({}, this.user, 'https://gitlab.com/gp-test-group/gp-test-project/-/tree/issue-%23123');
+        expect(result).to.deep.include({
+            "ref": "issue-#123",
+            "refType": "branch",
+            "path": "",
+            "revision": "8b389233c0a3a55a5cd75f89d2c96761420bf2c8",
+            "isFile": false,
+            "repository": {
+                "host": "gitlab.com",
+                "owner": "gp-test-group",
+                "name": "gp-test-project",
+                "cloneUrl": "https://gitlab.com/gp-test-group/gp-test-project.git",
+                "defaultBranch": "master",
+                "private": false
+            },
+            "title": "gp-test-group/gp-test-project - issue-#123"
+        })
+    }
+
+    @test public async testTreeContextBranchWithAndSign() {
+        const result = await this.parser.handle({}, this.user, 'https://gitlab.com/gp-test-group/gp-test-project/-/tree/another&amp;branch');
+        expect(result).to.deep.include({
+            "ref": "another&amp;branch",
+            "refType": "branch",
+            "path": "",
+            "revision": "8b389233c0a3a55a5cd75f89d2c96761420bf2c8",
+            "isFile": false,
+            "repository": {
+                "host": "gitlab.com",
+                "owner": "gp-test-group",
+                "name": "gp-test-project",
+                "cloneUrl": "https://gitlab.com/gp-test-group/gp-test-project.git",
+                "defaultBranch": "master",
+                "private": false
+            },
+            "title": "gp-test-group/gp-test-project - another&amp;branch"
+        })
+    }
+
     @test public async testEmptyProject() {
         const result = await this.parser.handle({}, this.user, 'https://gitlab.com/gp-test-group/gp-test-empty-project');
         expect(result).to.deep.include({
diff --git a/components/server/src/gitlab/gitlab-context-parser.ts b/components/server/src/gitlab/gitlab-context-parser.ts
index dbe1875aaf4bf2..6e98d524181d08 100644
--- a/components/server/src/gitlab/gitlab-context-parser.ts
+++ b/components/server/src/gitlab/gitlab-context-parser.ts
@@ -72,7 +72,9 @@ export class GitlabContextParser extends AbstractContextParser implements IConte
     public async parseURL(user: User, contextUrl: string): Promise<URLParts> {
         var { host, owner, repoName, moreSegments, searchParams } = await super.parseURL(user, contextUrl);
         // TODO: we remove the /-/ in the path in the next line as quick fix for #3809 -- improve this in the long term
-        const segments = [owner, repoName, ...moreSegments.filter(s => s !== '-')];
+        const segments = [owner, repoName, ...moreSegments.filter(s => s !== '-')]
+            // Replace URL encoded '#' sign. Don't use decodeURI() because GitLab seems to be inconsistent in what needs to be decoded and what not.
+            .map(x => x.replace(/%23/g, '#'));
         var moreSegmentsStart: number = 2;
         /*
             We cannot deduce the namespace (aka `owner`) and project name (aka `repoName`) from the URI with certainty.