Skip to content

Commit ce5f85e

Browse files
committed
update logger msg; use lower case for tool name
Signed-off-by: Sylwester Piskozub <sylwesterpiskozub@gmail.com>
1 parent 68c06cb commit ce5f85e

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

pkg/attestation/crafter/materials/gitleaks.go

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ func (i *GitleaksReportCrafter) Craft(ctx context.Context, filePath string) (*ap
5555

5656
// if list is empty. It's ambiguous, but we accept it
5757
if len(findings) == 0 {
58-
i.logger.Debug().Msg("Accepting an empty report. Make sure it's a valid Gitleaks report")
58+
i.logger.Debug().Msg("Accepting an empty report.")
5959
} else {
6060
finding := findings[0]
6161
// All gitleaks findings have a Fingerprint field. If this doesn't have it, it's not a gitleaks report
@@ -78,9 +78,9 @@ func (i *GitleaksReportCrafter) Craft(ctx context.Context, filePath string) (*ap
7878

7979
func (i *GitleaksReportCrafter) injectAnnotations(m *api.Attestation_Material) {
8080
// Gitleaks doesn't include version information in the JSON output
81-
// Set the tool name annotation
81+
// Set the tool name annotation https://github.com/gitleaks/gitleaks/blob/master/report/constants.go#L4
8282
if m.Annotations == nil {
8383
m.Annotations = make(map[string]string)
8484
}
85-
m.Annotations[AnnotationToolNameKey] = "Gitleaks"
85+
m.Annotations[AnnotationToolNameKey] = "gitleaks"
8686
}

0 commit comments

Comments
 (0)