fix: remove debugging from semgrep parser

BacchusJackson · BacchusJackson · commit 728b7aac8fc6 · 2024-05-15T21:37:32.000-05:00
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -5,6 +5,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [0.6.2] - 2024-05-15
+
+### Changed
+
+- Remove debug information from semgrep decoding
+
 ## [0.6.1] - 2024-05-15
 
 ### Changed
diff --git a/pkg/artifacts/v1/semgrep.go b/pkg/artifacts/v1/semgrep.go
@@ -2,7 +2,6 @@ package artifacts
 
 import (
 	"fmt"
-	"log/slog"
 	"strings"
 )
 
@@ -70,7 +69,6 @@ func (s *SemgrepResults) ShortCheckID() string {
 }
 
 func (s *SemgrepMetadata) OwaspIDs() string {
-	slog.Info(s.Shortlink, "type", fmt.Sprintf("%T", s.Owasp))
 	switch v := s.Owasp.(type) {
 	case string:
 		return v
diff --git a/test/semgrep-sast-report.json b/test/semgrep-sast-report.json

Original file line number	Diff line number	Diff line change
`@@ -2,7 +2,6 @@ package artifacts`
`2`	`2`
`3`	`3`	`import (`
`4`	`4`	`"fmt"`
`5`		`- "log/slog"`
`6`	`5`	`"strings"`
`7`	`6`	`)`
`8`	`7`
`@@ -70,7 +69,6 @@ func (s *SemgrepResults) ShortCheckID() string {`
`70`	`69`	`}`
`71`	`70`
`72`	`71`	`func (s *SemgrepMetadata) OwaspIDs() string {`
`73`		`- slog.Info(s.Shortlink, "type", fmt.Sprintf("%T", s.Owasp))`
`74`	`72`	`switch v := s.Owasp.(type) {`
`75`	`73`	`case string:`
`76`	`74`	`return v`