Fixes naming error for default fallbacks in harbor scanner matchers

uselagoon · Oct 23, 2020 · 3d61f61 · 3d61f61
1 parent 982cfe0
commit 3d61f61
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 8 deletions.
diff --git a/services/api/src/resources/problem/resolvers.ts b/services/api/src/resources/problem/resolvers.ts
@@ -199,7 +199,7 @@ export const addProblemHarborScanMatch = async (
       description,
       defaultLagoonProject,
       defaultLagoonEnvironment,
-      defaultLagoonServiceName,
+      defaultLagoonService,
       regex
     },
   },
@@ -219,7 +219,7 @@ export const addProblemHarborScanMatch = async (
         description,
         default_lagoon_project: defaultLagoonProject,
         default_lagoon_environment: defaultLagoonEnvironment,
-        default_lagoon_service_name: defaultLagoonServiceName,
+        default_lagoon_service_name: defaultLagoonService,
         regex
       }
     ),

diff --git a/services/api/src/resources/problem/sql.ts b/services/api/src/resources/problem/sql.ts
@@ -24,7 +24,7 @@ const standardProblemHarborScanMatchReturn = {
     description: 'description',
     default_lagoon_project: 'defaultLagoonProject',
     default_lagoon_environment: 'defaultLagoonEnvironment',
-    default_lagoon_service_name: 'defaultLagoonServiceName',
+    default_lagoon_service: 'defaultLagoonServiceName',
     regex: 'regex'
 };
 

diff --git a/services/webhooks2tasks/src/handlers/problems/harborScanningCompleted.ts b/services/webhooks2tasks/src/handlers/problems/harborScanningCompleted.ts
@@ -175,11 +175,10 @@ const matchRepositoryAgainstPatterns = (repoFullName, matchPatterns = []) => {
   }
 
   const matchPatternDetails = matchingRes.pop() || DEFAULT_REPO_DETAILS_MATCHER;
-
   const {
-    lagoonProjectName = matchPatternDetails.defaultProjectName,
-    lagoonEnvironmentName = matchPatternDetails.defaultEnvironmentName,
-    lagoonServiceName = matchPatternDetails.defaultServiceName,
+    lagoonProjectName = matchPatternDetails.defaultLagoonProject,
+    lagoonEnvironmentName = matchPatternDetails.defaultLagoonEnvironment,
+    lagoonServiceName = matchPatternDetails.defaultLagoonService,
   } = extractRepositoryDetailsGivenRegex(repoFullName, matchPatternDetails.regex);
 
   return {lagoonProjectName, lagoonEnvironmentName, lagoonServiceName};
@@ -190,7 +189,7 @@ const generateRegex = R.memoizeWith(R.identity, re => new RegExp(re));
 const extractRepositoryDetailsGivenRegex = (repoFullName, pattern = DEFAULT_REPO_DETAILS_REGEX) => {
   const re = generateRegex(pattern);
   const match = re.exec(repoFullName);
-  return match.groups;
+  return match.groups || {};
 }
 
 const generateWebhookData = (