fix gitlab init

Author: msukkari

packages/backend/src/ee/repoPermissionSyncer.ts

@@ -177,7 +177,7 @@ export class RepoPermissionSyncer {
 
         const accountIds = await (async () => {
             if (repo.external_codeHostType === 'github') {
-                const isGitHubCloud = credentials.hostUrl ? new URL(credentials.hostUrl).hostname === GITHUB_CLOUD_HOSTNAME : false;
+                const isGitHubCloud = credentials.hostUrl ? new URL(credentials.hostUrl).hostname === GITHUB_CLOUD_HOSTNAME : true;
                 const { octokit } = await createOctokitFromToken({
                     token: credentials.token,
                     url: isGitHubCloud ? undefined : credentials.hostUrl,

packages/backend/src/github.ts

@@ -47,7 +47,7 @@ const isHttpError = (error: unknown, status: number): boolean => {
 }
 
 export const createOctokitFromToken = async ({ token, url }: { token?: string, url?: string }): Promise<{ octokit: Octokit, isAuthenticated: boolean }> => {
-    const isGitHubCloud = url ? new URL(url).hostname === GITHUB_CLOUD_HOSTNAME : false;
+    const isGitHubCloud = url ? new URL(url).hostname === GITHUB_CLOUD_HOSTNAME : true;
     const octokit = new Octokit({
         auth: token,
         ...(url && !isGitHubCloud ? {

packages/backend/src/gitlab.ts

@@ -12,9 +12,9 @@ const logger = createLogger('gitlab');
 export const GITLAB_CLOUD_HOSTNAME = "gitlab.com";
 
 export const createGitLabFromPersonalAccessToken = async ({ token, url }: { token?: string, url?: string }) => {
-    const isGitLabCloud = url ? new URL(url).hostname === GITLAB_CLOUD_HOSTNAME : false;
+    const isGitLabCloud = url ? new URL(url).hostname === GITLAB_CLOUD_HOSTNAME : true;
     return new Gitlab({
-        token,
+        ...(token ? { token } : {}),
         ...(isGitLabCloud ? {} : {
             host: url,
         }),
@@ -23,9 +23,9 @@ export const createGitLabFromPersonalAccessToken = async ({ token, url }: { toke
 }
 
 export const createGitLabFromOAuthToken = async ({ oauthToken, url }: { oauthToken?: string, url?: string }) => {
-    const isGitLabCloud = url ? new URL(url).hostname === GITLAB_CLOUD_HOSTNAME : false;
+    const isGitLabCloud = url ? new URL(url).hostname === GITLAB_CLOUD_HOSTNAME : true;
     return new Gitlab({
-        oauthToken,
+        ...(oauthToken ? { oauthToken } : {}),
         ...(isGitLabCloud ? {} : {
             host: url,
         }),
@@ -99,7 +99,7 @@ export const getGitLabReposFromConfig = async (config: GitlabConnectionConfig) =
                 const status = e?.cause?.response?.status;
                 if (status !== undefined) {
                     const warning = `GitLab API returned ${status}`
-                    logger.warning(warning);
+                    logger.warn(warning);
                     return {
                         type: 'warning' as const,
                         warning
@@ -139,7 +139,7 @@ export const getGitLabReposFromConfig = async (config: GitlabConnectionConfig) =
                 const status = e?.cause?.response?.status;
                 if (status !== undefined) {
                     const warning = `GitLab API returned ${status}`
-                    logger.warning(warning);
+                    logger.warn(warning);
                     return {
                         type: 'warning' as const,
                         warning
@@ -177,7 +177,7 @@ export const getGitLabReposFromConfig = async (config: GitlabConnectionConfig) =
                 const status = e?.cause?.response?.status;
                 if (status !== undefined) {
                     const warning = `GitLab API returned ${status}`
-                    logger.warning(warning);
+                    logger.warn(warning);
                     return {
                         type: 'warning' as const,
                         warning