@ibm-cloud/cd-tools 1.2.5 → 1.3.0

package/cmd/utils/requests.js

@@ -302,8 +302,8 @@ async function getGitOAuth(bearer, targetRegion, gitId) {
     switch (response.status) {
         case 200:
             return response.data?.access_token;
-        case 500:
-            throw Error(response.data?.authorizationURI);
+        case 401:
+            throw Error(response.data?.authorizationURI ?? 'Get git OAuth failed');
         default:
             throw Error('Get git OAuth failed');
     }
@@ -332,14 +332,13 @@ async function getGritUserProject(privToken, region, user, projectName) {
 }
 
 async function getGritGroup(privToken, region, groupName) {
-    const url = `https://${region}.git.cloud.ibm.com/api/v4/groups`
+    const url = `https://${region}.git.cloud.ibm.com/api/v4/groups/${groupName}`
     const options = {
         url: url,
         method: 'GET',
         headers: {
             'PRIVATE-TOKEN': privToken
         },
-        params: { simple: true, search: groupName },
         validateStatus: () => true
     };
     const response = await axios(options);

package/cmd/utils/terraform.js

@@ -146,6 +146,13 @@ async function setupTerraformFiles({ token, srcRegion, targetRegion, targetTag,
         if (isCompact || resourceName === 'ibm_cd_toolchain_tool_hostedgit') {
             if (newTfFileObj['resource']['ibm_cd_toolchain_tool_hostedgit']) {
                 for (const [k, v] of Object.entries(newTfFileObj['resource']['ibm_cd_toolchain_tool_hostedgit'])) {
+                    try {
+                        newTfFileObj['resource']['ibm_cd_toolchain_tool_hostedgit'][k]['parameters'][0]['auth_type'] = 'oauth';
+                        delete newTfFileObj['resource']['ibm_cd_toolchain_tool_hostedgit'][k]['parameters'][0]['api_token'];
+                    } catch {
+                        // do nothing
+                    }
+
                     try {
                         const thisUrl = v['initialization'][0]['repo_url'];
                         if (thisUrl in gritMapping) {
@@ -163,6 +170,10 @@ async function setupTerraformFiles({ token, srcRegion, targetRegion, targetTag,
                         } else {
                             // prompt user
                             const validateGritUrlPrompt = async (str) => {
+                                if (!str) {
+                                    logger.print('Skipping... (URL will remain unchanged in the generatedTerraform configuration)');
+                                    return '';
+                                }
                                 const newUrl = `https://${targetRegion}.git.cloud.ibm.com/${str}.git`;
                                 if (usedGritUrls.has(newUrl)) throw Error(`"${newUrl}" has already been used in another mapping entry`);
                                 return validateGritUrl(token, targetRegion, str, false);
@@ -170,15 +181,17 @@ async function setupTerraformFiles({ token, srcRegion, targetRegion, targetTag,
 
                             if (!firstGritPrompt) {
                                 firstGritPrompt = true;
-                                logger.print('Please enter the new URLs for the following GRIT tool(s):\n');
+                                logger.print('Please enter the new URLs for the following GRIT tool(s) (or submit empty input to skip):\n');
                             }
 
                             const newRepoSlug = await promptUserInput(`Old URL: ${thisUrl.slice(0, thisUrl.length - 4)}\nNew URL: https://${targetRegion}.git.cloud.ibm.com/`, '', validateGritUrlPrompt);
 
-                            newUrl = `https://${targetRegion}.git.cloud.ibm.com/${newRepoSlug}.git`;
-                            newTfFileObj['resource']['ibm_cd_toolchain_tool_hostedgit'][k]['initialization'][0]['repo_url'] = newUrl;
-                            attemptAddUsedGritUrl(newUrl);
-                            gritMapping[thisUrl] = newUrl;
+                            if (newRepoSlug) {
+                                newUrl = `https://${targetRegion}.git.cloud.ibm.com/${newRepoSlug}.git`;
+                                newTfFileObj['resource']['ibm_cd_toolchain_tool_hostedgit'][k]['initialization'][0]['repo_url'] = newUrl;
+                                attemptAddUsedGritUrl(newUrl);
+                                gritMapping[thisUrl] = newUrl;
+                            }
                         }
                     }
                     catch (e) {

package/cmd/utils/validate.js

@@ -145,6 +145,7 @@ async function validateTools(token, tcId, region, skipPrompt) {
     const nonConfiguredTools = [];
     const toolsWithHashedParams = [];
     const patTools = [];
+    const gheTools = [];
     const classicPipelines = [];
 
     for (const tool of allTools.tools) {
@@ -195,14 +196,24 @@ async function validateTools(token, tcId, region, skipPrompt) {
                 url: toolUrl
             });
         }
-        else if (tool.tool_type_id === 'pipeline' && tool.parameters?.type === 'classic') { // Check for Classic pipelines
+
+        if (tool.tool_type_id === 'github_integrated') {
+            gheTools.push({
+                tool_name: toolName,
+                type: tool.tool_type_id,
+                url: toolUrl
+            });
+        }
+
+        if (tool.tool_type_id === 'pipeline' && tool.parameters?.type === 'classic') { // Check for Classic pipelines
             classicPipelines.push({
                 tool_name: toolName,
                 type: 'classic pipeline',
                 url: toolUrl
             });
         }
-        else if (['githubconsolidated', 'github_integrated', 'gitlab', 'hostedgit'].includes(tool.tool_type_id) && (tool.parameters?.auth_type === '' || tool.parameters?.auth_type === 'oauth')) { // Skip secret check iff it's GitHub/GitLab/GRIT integration with OAuth
+
+        if (['githubconsolidated', 'github_integrated', 'gitlab', 'hostedgit'].includes(tool.tool_type_id) && (tool.parameters?.auth_type === '' || tool.parameters?.auth_type === 'oauth')) { // Skip secret check iff it's GitHub/GitLab/GRIT integration with OAuth
             continue;
         }
         else {
@@ -240,7 +251,7 @@ async function validateTools(token, tcId, region, skipPrompt) {
             }
         }
     }
-    const hasInvalidConfig = nonConfiguredTools.length > 0 || patTools.length > 0 || toolsWithHashedParams.length > 0;
+    const hasInvalidConfig = nonConfiguredTools.length > 0 || toolsWithHashedParams.length > 0;
 
     if (classicPipelines.length > 0) {
         logger.failSpinner('Unsupported tools found!');
@@ -257,10 +268,15 @@ async function validateTools(token, tcId, region, skipPrompt) {
     }
 
     if (patTools.length > 0) {
-        logger.warn('Warning! The following GRIT integration(s) are using auth_type "pat", please switch to auth_type "oauth" before proceeding: \n', LOG_STAGES.setup, true);
+        logger.warn('Warning! The following GRIT integration(s) with auth_type "pat" are unsupported during migration and will automatically be converted to auth_type "oauth": \n', LOG_STAGES.setup, true);
         logger.table(patTools);
     }
 
+    if (gheTools.length > 0) {
+        logger.warn('Warning! The following legacy GHE integration(s) are unsupported during migration will automatically be converted to equivalent GitHub integrations: \n', LOG_STAGES.setup, true);
+        logger.table(gheTools);
+    }
+
     if (toolsWithHashedParams.length > 0) {
         logger.warn('Warning! The following tools contain secrets that cannot be migrated, please use the \'check-secrets\' command to export the secrets: \n', LOG_STAGES.setup, true);
         logger.table(toolsWithHashedParams);
@@ -353,12 +369,10 @@ async function validateOAuth(token, tools, targetRegion, skipPrompt) {
                     type: tool.tool_type_id + (isGHE ? ' (GHE)' : ''),
                     link: authorizeUrl?.message != 'Get git OAuth failed' ? 'See link below' : 'Get git OAuth failed',
                 })
-                if (authorizeUrl?.message != 'Get git OAuth failed') {
-                    if (isGHE) {
-                        oauthLinks.push({ type: 'githubconsolidated (GHE)', link: authorizeUrl?.message });
-                    } else {
-                        oauthLinks.push({ type: tool.tool_type_id, link: authorizeUrl?.message ?? 'Could not get OAuth link' });
-                    }
+                if (isGHE) {
+                    oauthLinks.push({ type: 'githubconsolidated (GHE)', link: authorizeUrl?.message });
+                } else {
+                    oauthLinks.push({ type: tool.tool_type_id, link: authorizeUrl?.message });
                 }
             }
         }
@@ -376,7 +390,7 @@ async function validateOAuth(token, tools, targetRegion, skipPrompt) {
 
         if (oauthLinks.length > 0) logger.print('Authorize using the following links:\n');
         oauthLinks.forEach((o) => {
-            if (o.link === 'Could not get OAuth link') hasFailedLink = true;
+            if (o.link === 'Get git OAuth failed') hasFailedLink = true;
             logger.print(`${o.type}: \x1b[36m${o.link}\x1b[0m\n`);
         });
 
@@ -447,8 +461,7 @@ async function validateGritUrl(token, region, url, validateFull) {
 
     // try group
     try {
-        const groupId = await getGritGroup(accessToken, region, urlStart);
-        await getGritGroupProject(accessToken, region, groupId, projectName);
+        await getGritGroupProject(accessToken, region, urlStart, projectName);
         return trimmed;
     } catch {
         throw Error('Provided GRIT url not found');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ibm-cloud/cd-tools",
-  "version": "1.2.5",
+  "version": "1.3.0",
   "description": "Tools and utilities for the IBM Cloud Continuous Delivery service and resources",
   "repository": {
     "type": "git",