fix: TFSec outputs the same results multiple times (#237)

antonbabenko · Oct 14, 2021 · 71f7c34 · 71f7c34
1 parent 3ae5eb0
commit 71f7c34
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/.pre-commit-hooks.yaml b/.pre-commit-hooks.yaml
@@ -79,6 +79,7 @@
 - id: terraform_tfsec
   name: Terraform validate with tfsec
   description: Static analysis of Terraform templates to spot potential security issues.
+  require_serial: true
   entry: terraform_tfsec.sh
   language: script
 

diff --git a/terraform_tfsec.sh b/terraform_tfsec.sh
@@ -6,13 +6,13 @@ main() {
   parse_cmdline_ "$@"
 
   # propagate $FILES to custom function
-  tfsec_ "$ARGS" "$FILES"
+  tfsec_ "$ARGS" "${FILES[*]}"
 }
 
 tfsec_() {
   # consume modified files passed from pre-commit so that
   # tfsec runs against only those relevant directories
-  for file_with_path in $FILES; do
+  for file_with_path in ${FILES[*]}; do
     file_with_path="${file_with_path// /__REPLACED__SPACE__}"
     paths[index]=$(dirname "$file_with_path")