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

logging Status with JSON format #874

Closed
BalloonWen opened this issue Dec 17, 2020 · 2 comments
Closed

logging Status with JSON format #874

BalloonWen opened this issue Dec 17, 2020 · 2 comments
Assignees

Comments

@BalloonWen
Copy link
Contributor

logging status info with JSON format instead of Stringified JSON
eg. logger.error("ValidationError:" + status.toString());
status.toString() is pure JSON, but with prefix ValidationError, it cannot be recognized by logging framework.

@BalloonWen BalloonWen self-assigned this Dec 17, 2020
BalloonWen added a commit that referenced this issue Dec 17, 2020
- separate logging status.toString()
stevehu pushed a commit that referenced this issue Dec 17, 2020
- separate logging status.toString()
@stevehu
Copy link
Contributor

stevehu commented Dec 17, 2020

@stevehu
Copy link
Contributor

stevehu commented Dec 17, 2020

For some users, they are using the JSON appenders so that the logs can be injected into Splunk. Mixing string and JSON in the same logging statement will cause issues for those appenders. Thank you @BalloonWen for your help.

@stevehu stevehu closed this as completed Dec 22, 2020
younggwon1 pushed a commit to younggwon1/light-4j that referenced this issue Feb 10, 2024
- separate logging status.toString()
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

No branches or pull requests

2 participants