Exception handling on loggers

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

• Close this issue

Summary

We should add exception handling (Gitlab::Sentry.track_exception(e)) to the loggers that need them.

See this discussion.

Improvements

An unhandled exception in a logger should not bring down an entire request. Handling these errors insulates end-users from flaky failures (that have nothing to do with them or their requests) while also alerting the dev team so we can iterate with the right information in the right channels.

Risks

Not sure of any.

Involved components

Potentially *_logger.rb

Optional: Intended side effects

Optional: Missing test coverage