internal/middleware: downgrade quota timeout to warning
Change-Id: Idea6b410ec906441313f1ddd75bcb615de234679
Reviewed-on: https://go-review.googlesource.com/c/pkgsite/+/276152
Trust: Jonathan Amsterdam <jba@google.com>
Run-TryBot: Jonathan Amsterdam <jba@google.com>
TryBot-Result: kokoro <noreply+kokoro@google.com>
Reviewed-by: Julie Qiu <julie@golang.org>
diff --git a/internal/middleware/quota.go b/internal/middleware/quota.go
index b482abc..effcf77 100644
--- a/internal/middleware/quota.go
+++ b/internal/middleware/quota.go
@@ -177,14 +177,12 @@
rrateKey := string(mac.Sum(nil))
res, err := rrate.NewLimiter(client.WithTimeout(15*time.Millisecond)).Allow(ctx, rrateKey, rrate.PerSecond(qps))
if err != nil {
- log.Errorf(ctx, "quota: redis limiter: %v", err)
- if errors.Is(err, context.DeadlineExceeded) {
- return false, "timeout"
- }
var nerr *net.OpError
- if errors.As(err, &nerr) && nerr.Timeout() {
+ if errors.Is(err, context.DeadlineExceeded) || (errors.As(err, &nerr) && nerr.Timeout()) {
+ log.Warningf(ctx, "quota: redis limiter: %v", err)
return false, "timeout"
}
+ log.Errorf(ctx, "quota: redis limiter: %v", err)
return false, "error"
}
if res.Allowed > 0 {
