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

feat: Support json logging #764

Merged
merged 1 commit into from
Jan 21, 2025
Merged

feat: Support json logging #764

merged 1 commit into from
Jan 21, 2025

Conversation

ryanfaircloth
Copy link
Contributor

No description provided.

@gaul gaul merged commit 7239ed4 into gaul:master Jan 21, 2025
3 checks passed
@gaul
Copy link
Owner

gaul commented Jan 21, 2025

Thank you for your contribution @ryanfaircloth! Are these structured logs better for Loggly or some other log aggregator use case?

@ryanfaircloth
Copy link
Contributor Author

Most log systems actually will prefer it Splunk, CrowdStrike logscale and others. In particular it helps with multiline logoutput for container deployments because its difficult to assign config to file readers to deal with start and end events by using json the agents and destination systems at least don't have to get the start/end right

@gaul
Copy link
Owner

gaul commented Jan 21, 2025

Thanks for the explanation! Please let me know if there is anything S3Proxy could do to fit better into your environment.

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

Successfully merging this pull request may close these issues.

2 participants