feat(misconf): log causes of HCL file parsing errors

Signed-off-by: nikpivkin <[email protected]>

pkg/iac/scanners/terraform/parser/parser.go

@@ -1,8 +1,10 @@
 package parser
 
 import (
+	"bufio"
 	"context"
 	"errors"
+	"fmt"
 	"io"
 	"io/fs"
 	"os"
@@ -163,6 +165,14 @@ func (p *Parser) ParseFS(ctx context.Context, dir string) error {
 			if p.stopOnHCLError {
 				return err
 			}
+			var diags hcl.Diagnostics
+			if errors.As(err, &diags) {
+				errc := p.showParseErrors(p.moduleFS, path, diags)
+				if errc == nil {
+					continue
+				}
+				p.logger.Error("Failed to get the causes of the parsing error", log.Err(errc))
+			}
 			p.logger.Error("Error parsing file", log.FilePath(path), log.Err(err))
 			continue
 		}
@@ -171,6 +181,46 @@ func (p *Parser) ParseFS(ctx context.Context, dir string) error {
 	return nil
 }
 
+func (p *Parser) showParseErrors(fsys fs.FS, path string, diags hcl.Diagnostics) error {
+	file, err := fsys.Open(path)
+	if err != nil {
+		return fmt.Errorf("failed to read file: %w", err)
+	}
+	defer file.Close()
+
+	for _, diag := range diags {
+		if subj := diag.Subject; subj != nil {
+			lines, err := readLinesFromFile(file, subj.Start.Line, subj.End.Line)
+			if err != nil {
+				return err
+			}
+
+			cause := strings.Join(lines, "\n")
+			p.logger.Error("Error parsing file", log.FilePath(path),
+				log.String("cause", cause), log.Err(diag))
+		}
+	}
+
+	return nil
+}
+
+func readLinesFromFile(f io.Reader, from, to int) ([]string, error) {
+	scanner := bufio.NewScanner(f)
+	rawLines := make([]string, 0, to-from+1)
+
+	for lineNum := 0; scanner.Scan() && lineNum < to; lineNum++ {
+		if lineNum >= from-1 {
+			rawLines = append(rawLines, scanner.Text())
+		}
+	}
+
+	if err := scanner.Err(); err != nil {
+		return nil, fmt.Errorf("failed to scan file: %w", err)
+	}
+
+	return rawLines, nil
+}
+
 var ErrNoFiles = errors.New("no files found")
 
 func (p *Parser) Load(ctx context.Context) (*evaluator, error) {

pkg/iac/scanners/terraform/parser/parser_test.go

@@ -1884,3 +1884,24 @@ variable "baz" {}
 	assert.Contains(t, buf.String(), "Variable values was not found in the environment or variable files.")
 	assert.Contains(t, buf.String(), "variables=\"foo\"")
 }
+
+func TestLogParseErrors(t *testing.T) {
+	var buf bytes.Buffer
+	slog.SetDefault(slog.New(log.NewHandler(&buf, nil)))
+
+	src := `resource "aws-s3-bucket" "name" {
+  bucket = <
+}`
+
+	fsys := fstest.MapFS{
+		"main.tf": &fstest.MapFile{
+			Data: []byte(src),
+		},
+	}
+
+	parser := New(fsys, "")
+	err := parser.ParseFS(context.TODO(), ".")
+	require.NoError(t, err)
+
+	assert.Contains(t, buf.String(), `cause="  bucket = <"`)
+}