mozilla-services / shavar

Tracking Protection update service for Firefox based on Safe Browsing protocol
Mozilla Public License 2.0
20 stars 10 forks source link

Deprecate logger.error to .info to reduce Sentry events #166

Closed say-yawn closed 1 year ago

say-yawn commented 1 year ago

Some of these events are NOT necessarily and error, more like a warning and can be deprecated to warning or info:

say-yawn commented 1 year ago

I decided to keep the No chunk file found ... as an error because this is a valid error that can happen if the Shavar server config is looking for files that is supposed to exist but is not yet in S3 which is the result of this error. This old Sentry errors show that the event stops after a while which shows that the lists were available in S3 and if it continued to not exist in S3 this is an error we should address.