Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

help reducing log pollution #29

Open
vincentcr opened this issue Aug 22, 2023 · 2 comments
Open

help reducing log pollution #29

vincentcr opened this issue Aug 22, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@vincentcr
Copy link

Summary

It would be great if the Amplitude experiment SDK config accepted a logger parameter so we could format error logs coming out of Amplitude to conform to our logging system.

Motivations

We use Datadog and the rest of our logs are emitted as JSON lines. Amplitude dumps stack traces directly to stdout and DD treats each line of the trace as a separate log entry. This not only pollutes our logs but makes hard to analyze any potential issue with Amplitude in particular.

This issue would be resolved if we could configure the logging interface the Amplitude SDK uses to emit errors. Launch Darkly for example has a logger parameter in its init function.

@vincentcr vincentcr added the enhancement New feature or request label Aug 22, 2023
@bgiori
Copy link
Collaborator

bgiori commented Aug 22, 2023

@vincentcr Thanks for the feedback! Agreed. I've added this to our backlog.

@calm-mlin
Copy link

Any update on this? Looks like Amplitude might have had an outage recently and our logs got severely spammed by the console.xxx() used by this library. It's fairly common to have an agent that reads stdout and treats each lines as a single log entry.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants