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
Add documentation for dump handler #30
Conversation
content/concern/dump.md
Outdated
@@ -6,10 +6,142 @@ categories: [concerns] | |||
keywords: [] | |||
aliases: [] | |||
toc: false | |||
draft: false | |||
draft: true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
draft should be false
content/concern/dump.md
Outdated
|
||
This handler is not suggested to be used on production because it will impact the overall performance. | ||
|
||
The default request/response log will be dump.log config. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Or whatever's configured in logback.xml right?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yes. It won't be automatically generated in logback.xml, so there is not "default" thing. User needs to add it manually. BTW, do I need to add dump.log in logback.xml when doing the code generation like "audit.log"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@BalloonWen Good point, i hadn't considered that. Yeah i think codegen should include the dump.log reference. @stevehu what do you think?
content/concern/dump.md
Outdated
|
||
This is a generic dump handler may be useful to troubleshooting in developing/testing environment. | ||
|
||
This handler is not suggested to be used on production because it will impact the overall performance. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this is already said on line 13.
-documentation for networknt/light-4j#328