fix(terraform): check line validity (#12102)

renovatebot · Oct 10, 2021 · da1b7d5 · da1b7d5
1 parent f74404b
commit da1b7d5
Showing 1 changed file with 15 additions and 14 deletions.
diff --git a/lib/manager/terraform/providers.ts b/lib/manager/terraform/providers.ts
@@ -36,24 +36,25 @@ export function extractTerraformProvider(
     }
 
     const line = lines[lineNumber];
-    // `{` will be counted wit +1 and `}` with -1. Therefore if we reach braceCounter == 0. We have found the end of the terraform block
-    const openBrackets = (line.match(/\{/g) || []).length;
-    const closedBrackets = (line.match(/\}/g) || []).length;
-    braceCounter = braceCounter + openBrackets - closedBrackets;
+    if (line) {
+      // `{` will be counted wit +1 and `}` with -1. Therefore if we reach braceCounter == 0. We have found the end of the terraform block
+      const openBrackets = (line.match(/\{/g) || []).length;
+      const closedBrackets = (line.match(/\}/g) || []).length;
+      braceCounter = braceCounter + openBrackets - closedBrackets;
 
-    // only update fields inside the root block
-    if (braceCounter === 1) {
-      const kvMatch = keyValueExtractionRegex.exec(line);
-      if (kvMatch) {
-        if (kvMatch.groups.key === 'version') {
-          dep.currentValue = kvMatch.groups.value;
-        } else if (kvMatch.groups.key === 'source') {
-          dep.managerData.source = kvMatch.groups.value;
-          dep.managerData.sourceLine = lineNumber;
+      // only update fields inside the root block
+      if (braceCounter === 1) {
+        const kvMatch = keyValueExtractionRegex.exec(line);
+        if (kvMatch) {
+          if (kvMatch.groups.key === 'version') {
+            dep.currentValue = kvMatch.groups.value;
+          } else if (kvMatch.groups.key === 'source') {
+            dep.managerData.source = kvMatch.groups.value;
+            dep.managerData.sourceLine = lineNumber;
+          }
         }
       }
     }
-
     lineNumber += 1;
   } while (braceCounter !== 0);
   deps.push(dep);