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

Fix/#876 LightHttpHandler is getting auditOnError and auditStackTrace config properly. #882

Merged
merged 3 commits into from Dec 24, 2020

Conversation

BalloonWen
Copy link
Contributor

now LightHttpHandler will get auditOnError and auditStackTrace config properly.
Now will only audit response body when auditOnError is ON
Now will only audit stacktrace when auditStackTrace is ON

- now LightHttpHandler is getting auditOnError and auditStackTrace correctly.
- added auditConfig check
- refactored tests for audit
- adjusted tests for audit
- removed unused handler.yml
- updated comment
@BalloonWen BalloonWen added bug Issue: Bug 2.0.x Merge to 2.0.x Branch labels Dec 22, 2020
@BalloonWen BalloonWen linked an issue Dec 22, 2020 that may be closed by this pull request
@stevehu stevehu merged commit 16e85b8 into master Dec 24, 2020
@stevehu stevehu deleted the fix/#876 branch December 24, 2020 02:30
younggwon1 pushed a commit to younggwon1/light-4j that referenced this pull request Feb 10, 2024
…tackTrace config properly. (networknt#882)

* issue networknt#876
- now LightHttpHandler is getting auditOnError and auditStackTrace correctly.

* issue - networknt#876
- added auditConfig check
- refactored tests for audit
- adjusted tests for audit
- removed unused handler.yml

* issue - networknt#876
- updated comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2.0.x Merge to 2.0.x Branch bug Issue: Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

auditOnError config is not properly set in LightHttpHandler
2 participants