openverse-api icon indicating copy to clipboard operation
openverse-api copied to clipboard

Add Slack functionality inside logger

Open dhruvkb opened this issue 2 years ago • 2 comments

Problem

In the ingestion_server codebase, calls to log functions like log.info and log.error are generally followed by sending the same information to Slack using the wrapper. This can be solved in a more elegant way by making Slack itself a logging destination within the Python logging framework.

Description

Integrating Slack with the logging configuration means that

  • we won't have to invoke Slack everywhere manually
  • Slack can make use of Pythonic log levels and other utilities like formatters

Alternatives

We can either roll with a homemade solution or use something like https://github.com/mathiasose/slacker_log_handler.

Implementation

  • [ ] 🙋 I would be interested in implementing this feature.

dhruvkb avatar May 21 '22 05:05 dhruvkb

Hi @dhruvkb , This issue seems to be a good initiation of my journey towards contribution to FOSS projects. Can you please assign this to me?

Akshat977 avatar Sep 09 '22 04:09 Akshat977

Sure @Akshat977, assigned to you! If you need help getting started, leave a comment here or you can join us in the #openverse channel in the Making WordPress Slack workspace.

dhruvkb avatar Sep 09 '22 06:09 dhruvkb