[server] properly parse ports of clone URLs

Otherwise this breaks the lookup of services mapped by host (including the port.) This is the quite apparent aftermath of enabling ports in SCM locations.

Author: AlexTugarev

components/server/src/repohost/repo-url.spec.ts

@@ -72,6 +72,16 @@ export class RepoUrlTest {
  repo: "yolo",
  });
  }
+
+ @test public parseScmCloneUrl_with_port() {
+ const testUrl = RepoURL.parseRepoUrl("https://foo.bar.com:12345/scm/proj/repoName.git");
+ expect(testUrl).to.deep.include({
+ host: "foo.bar.com:12345",
+ repoKind: "projects",
+ owner: "proj",
+ repo: "repoName",
+ });
+ }
 }
 
 module.exports = new RepoUrlTest();

components/server/src/repohost/repo-url.ts

@@ -10,7 +10,7 @@ export namespace RepoURL {
  repoUrl: string,
  ): { host: string; owner: string; repo: string; repoKind?: string } | undefined {
  const u = new URL(repoUrl);
- const host = u.hostname || "";
+ const host = u.host || "";
  const path = u.pathname || "";
  const segments = path.split("/").filter((s) => !!s); // e.g. [ 'gitpod-io', 'gitpod.git' ]
  if (segments.length === 2) {