ovhemert / pino-datadog

🌲A transport for pino that sends messages to DataDog
MIT License
36 stars 24 forks source link

Allow both global and log-level tags #67

Closed ephemer closed 2 years ago

ephemer commented 3 years ago

We have the need for global / configuration-level tags and log-level tags. In the current implementation that is not possible, because the per-log ddtags are ignored if ddtags is also sent as a property of params on the http request to the datadog API. That appears to be an issue with the API that we need to work around here.

I have tested this manually but will add unit tests if possible before submitting for review (hence the draft status). I will also look at updating the documentation when I do that.

Checklist

stale[bot] commented 2 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.