sorentwo / oban

💎 Robust job processing in Elixir, backed by modern PostgreSQL and SQLite3
https://getoban.pro
Apache License 2.0
3.17k stars 297 forks source link

Request to rename emitted log attribute for `[:oban, :notifier, :switch]` events. #1085

Closed punchcafe closed 1 month ago

punchcafe commented 1 month ago

Hello Again!

This is a small change request to make the default telemetry logger integrate nicer with Datadog.

The logs emitted in response to [:oban, :notifier, :switch] events have a status field, which is a reserved field for JSON logs with Datadog. The result was that all :switch events were being treated as critical logs and raising alerts. I imagine a lot of Oban users use Datadog so I thought it may be worth making this small tweak to avoid conflict.

Thanks, Luca