Skip to content

Commit

Permalink
remove lgtm, it does nothing
Browse files Browse the repository at this point in the history
Signed-off-by: Hayden Blauzvern <[email protected]>
  • Loading branch information
haydentherapper committed Jan 15, 2025
1 parent 424185a commit b548f71
Showing 1 changed file with 0 additions and 2 deletions.
2 changes: 0 additions & 2 deletions pkg/sharding/ranges.go
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ func NewLogRanges(ctx context.Context, logClient trillian.TrillianLogClient,
if err != nil {
return LogRanges{}, fmt.Errorf("creating range for active tree %d: %w", activeTreeID, err)
}
// lgtm [go/clear-text-logging]
log.Logger.Infof("Active log: %s", activeLog.String())

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information High

Sensitive data returned by an access to FileSignerPassword
flows to a logging call.

if inactiveShardsPath == "" {
Expand All @@ -97,7 +96,6 @@ func NewLogRanges(ctx context.Context, logClient trillian.TrillianLogClient,
ranges[i] = r
}
for i, r := range ranges {
// lgtm [go/clear-text-logging]
log.Logger.Infof("Inactive range %d: %s", i, r.String())

Check failure

Code scanning / CodeQL

Clear-text logging of sensitive information High

Sensitive data returned by an access to FileSignerPassword
flows to a logging call.
}

Expand Down

0 comments on commit b548f71

Please sign in to comment.