better logs for gitlab config sync

Author: msukkari

packages/backend/src/gitlab.ts

@@ -97,13 +97,10 @@ export const getGitLabReposFromConfig = async (config: GitlabConnectionConfig) =
                 logger.error(`Failed to fetch projects for group ${group}.`, e);
 
                 const status = e?.cause?.response?.status;
-                if (status === 404) {
-                    const warning = `Group ${group} not found or no access`;
-                    logger.warn(warning);
-                    return {
-                        type: 'warning' as const,
-                        warning
-                    };
+                if (status !== undefined) {
+                    logger.error(`HTTP status: ${status}`);
+                } else {
+                    logger.error(`HTTP status: undefined`);
                 }
                 throw e;
             }
@@ -135,13 +132,10 @@ export const getGitLabReposFromConfig = async (config: GitlabConnectionConfig) =
                 logger.error(`Failed to fetch projects for user ${user}.`, e);
 
                 const status = e?.cause?.response?.status;
-                if (status === 404) {
-                    const warning = `User ${user} not found or no access`;
-                    logger.warn(warning);
-                    return {
-                        type: 'warning' as const,
-                        warning
-                    };
+                if (status !== undefined) {
+                    logger.error(`HTTP status: ${status}`);
+                } else {
+                    logger.error(`HTTP status: undefined`);
                 }
                 throw e;
             }
@@ -171,14 +165,10 @@ export const getGitLabReposFromConfig = async (config: GitlabConnectionConfig) =
                 logger.error(`Failed to fetch project ${project}.`, e);
 
                 const status = e?.cause?.response?.status;
-
-                if (status === 404) {
-                    const warning = `Project ${project} not found or no access`;
-                    logger.warn(warning);
-                    return {
-                        type: 'warning' as const,
-                        warning
-                    };
+                if (status !== undefined) {
+                    logger.error(`HTTP status: ${status}`);
+                } else {
+                    logger.error(`HTTP status: undefined`);
                 }
                 throw e;
             }