Validate error loggings
Problem
When using error_log
for logging, it will automatically end up raising an exception in Sentry. Since we mostly use log_error
and rarely log_warn
, this leads to a lot of unnecessary exceptions that don't require the teams' intention. This is the most recent example of a validation error when someone tried to create a license manually in the Admin area. There are other kinds of these exceptions, e.g. when a customer tries to use an invalid or expired credit credit when purchasing something.
Examples
Proposal
Evaluate all usages of log_error
and convert the ones we don't actually need to raise an exception for to log_warn
.
Result
Reduces unnecessary raised exception and noise in the fulfillment status Slack channel.
Edited by Corinna Gogolok