fix: allow long remote URLs to be parsed

Author: sdzh-atlassian

package-lock.json

@@ -683,7 +683,7 @@
                     "when": "atlascode:isBitbucketCloudRepo && config.atlascode.bitbucket.enabled",
                     "group": "navigation"
                 }
-			],
+            ],
             "view/title": [
                 {
                     "command": "atlascode.rovodev.newChatSession",
@@ -1700,6 +1700,7 @@
         "p-queue": "^6.3.0",
         "p-settle": "^3.1.0",
         "p-timeout": "^3.2.0",
+        "parse-url": "^8.1.0",
         "prosemirror-commands": "^1.7.1",
         "prosemirror-dropcursor": "^1.8.2",
         "prosemirror-example-setup": "^1.2.3",

package.json

@@ -93,37 +93,42 @@ export class BitbucketContext extends Disposable {
         this._pullRequestCache.clear();
         this._repoMap.clear();
 
-        await Promise.all(
-            Container.siteManager.getSitesAvailable(ProductBitbucket).map(async (site) => {
-                try {
-                    const bbApi = await Container.clientManager.bbClient(site);
-                    const mirrorHosts = await bbApi.repositories.getMirrorHosts();
-                    this._mirrorsCache.setItem(site.host, mirrorHosts);
-                } catch {
-                    // log and ignore error
-                    Logger.debug('Failed to fetch mirror sites');
-                }
-            }),
-        );
+        try {
+            await Promise.all(
+                Container.siteManager.getSitesAvailable(ProductBitbucket).map(async (site) => {
+                    try {
+                        const bbApi = await Container.clientManager.bbClient(site);
+                        const mirrorHosts = await bbApi.repositories.getMirrorHosts();
+                        this._mirrorsCache.setItem(site.host, mirrorHosts);
+                    } catch {
+                        // log and ignore error
+                        Logger.debug('Failed to fetch mirror sites');
+                    }
+                }),
+            );
 
-        const repos = this.getAllRepositoriesRaw();
-        for (let i = 0; i < repos.length; i++) {
-            const repo: Repository = repos[i];
-            if (!repo.state.HEAD) {
-                Logger.debug(`JS-1324 Forcing updateModelState on ${repo.rootUri}`);
-                await repo.status();
-            }
-            if (repo.state.remotes.length > 0) {
-                this._repoMap.set(repo.rootUri.toString(), workspaceRepoFor(repo));
-            } else {
-                Logger.warn(`JS-1324 no remotes found for ${repo.rootUri}`);
+            const repos = this.getAllRepositoriesRaw();
+            for (let i = 0; i < repos.length; i++) {
+                const repo: Repository = repos[i];
+                if (!repo.state.HEAD) {
+                    Logger.debug(`JS-1324 Forcing updateModelState on ${repo.rootUri}`);
+                    await repo.status();
+                }
+                if (repo.state.remotes.length > 0) {
+                    this._repoMap.set(repo.rootUri.toString(), workspaceRepoFor(repo));
+                } else {
+                    Logger.warn(`JS-1324 no remotes found for ${repo.rootUri}`);
+                }
             }
-        }
 
-        const isBitbucketCloudRepo = this.getBitbucketCloudRepositories().length > 0;
-        this.setIsBitbucketCloudRepo(isBitbucketCloudRepo);
+            const isBitbucketCloudRepo = this.getBitbucketCloudRepositories().length > 0;
+            this.setIsBitbucketCloudRepo(isBitbucketCloudRepo);
 
-        this._onDidChangeBitbucketContext.fire();
+            this._onDidChangeBitbucketContext.fire();
+        } catch (err) {
+            err.subject_url = undefined; // remove potentially sensitive info
+            Logger.error(err, 'Error refreshing Bitbucket repositories');
+        }
     }
 
     private updateUsers(sites: DetailedSiteInfo[]) {

src/bitbucket/bbContext.ts

@@ -8,6 +8,11 @@ import { BitbucketApi, BitbucketSite, WorkspaceRepo } from './model';
 
 const bbServerRepoRegEx = new RegExp(/(?<type>users|projects)\/(?<owner>.*)\/repos/);
 
+// Override the max input length to support long Bitbucket Server URLs
+// The error is thrown in `parse-url` <- `git-up` <- `git-url-parse`, hence the odd modification here
+const parseUrl = require('parse-url');
+parseUrl.MAX_INPUT_LENGTH = 3000;
+
 export function parseGitUrl(url: string): gitUrlParse.GitUrl {
     const parsed = gitUrlParse(url);
     parsed.owner = parsed.owner.slice(parsed.owner.lastIndexOf('/') + 1);