Skip to content

Commit c40576d

Browse files
committed
remove unneeded PAT embed check
1 parent 2b46e8a commit c40576d

File tree

2 files changed

+0
-14
lines changed

2 files changed

+0
-14
lines changed

packages/backend/src/git.ts

Lines changed: 0 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,6 @@
11
import { CheckRepoActions, GitConfigScope, simpleGit, SimpleGitProgressEvent } from 'simple-git';
22
import { mkdir } from 'node:fs/promises';
33
import { env } from './env.js';
4-
import { doesHaveEmbeddedToken } from './utils.js';
54

65
type onProgressFn = (event: SimpleGitProgressEvent) => void;
76

@@ -28,10 +27,6 @@ export const cloneRepository = async (
2827
})
2928

3029
if (authHeader) {
31-
if (doesHaveEmbeddedToken(cloneUrl)) {
32-
throw new Error("Cannot use auth header when clone URL has embedded token");
33-
}
34-
3530
await git.clone(
3631
cloneUrl,
3732
path,
@@ -87,10 +82,6 @@ export const fetchRepository = async (
8782
})
8883

8984
if (authHeader) {
90-
if (doesHaveEmbeddedToken(cloneUrl)) {
91-
throw new Error("Cannot use auth header when clone URL has embedded token");
92-
}
93-
9485
await git.addConfig("http.extraHeader", authHeader);
9586
}
9687

packages/backend/src/utils.ts

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -241,8 +241,3 @@ const createGitCloneUrlWithToken = (cloneUrl: string, credentials: { username?:
241241
}
242242
return url.toString();
243243
}
244-
245-
export const doesHaveEmbeddedToken = (cloneUrl: string) => {
246-
const url = new URL(cloneUrl);
247-
return url.username || url.password;
248-
}

0 commit comments

Comments
 (0)