Skip to content

Commit

Permalink
Reduce redis rate limit construction logs from audit to info (#7516)
Browse files Browse the repository at this point in the history
Everything logged at the error level is implicitly given the audit tag
as well. That's not merited in this case, so downgrade these error logs
to be info logs instead.
  • Loading branch information
aarongable authored Jun 3, 2024
1 parent 6277644 commit f84050a
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion wfe2/wfe.go
Original file line number Diff line number Diff line change
Expand Up @@ -2057,7 +2057,7 @@ func (wfe *WebFrontEndImpl) newNewOrderLimitTransactions(regId int64, names []st
logTxnErr := func(err error, limit ratelimits.Name) {
// TODO(#5545): Once key-value rate limits are authoritative this log
// line should be removed in favor of returning the error.
wfe.log.Errf("constructing rate limit transaction for %s rate limit: %s", limit, err)
wfe.log.Infof("error constructing rate limit transaction for %s rate limit: %s", limit, err)
}

var transactions []ratelimits.Transaction
Expand Down

0 comments on commit f84050a

Please sign in to comment.