The API currently logs everything, including user errors, with a severity of Error. Logging user errors may be appropriate in a debugging scenario, but in the course of normal operations a call which failed to find any documents is a normal occurrence and logging it only serves to fill the logs with noise messages.
ESTIMATE TBD
What's the expected change?
Errors resulting from user inputs should be logged with a severity of Debug. (Errors resulting from system errors, e.g. search fails because Elasticsearch is offline, should continue to be logged as Error, perhaps even Critical.)
blairlearn
commented
1 year ago
Issue description
The API currently logs everything, including user errors, with a severity of
Error. Logging user errors may be appropriate in a debugging scenario, but in the course of normal operations a call which failed to find any documents is a normal occurrence and logging it only serves to fill the logs with noise messages.What's the expected change?
Debug. (Errors resulting from system errors, e.g. search fails because Elasticsearch is offline, should continue to be logged asError, perhaps evenCritical.)What's the current functionality?
What's the updated acceptance criteria?
Additional details / screenshot
Related Tickets