-
Notifications
You must be signed in to change notification settings - Fork 369
Avoid using System.err in EsapiPropertyManager #386
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
Comments
matthiaslarisch
added a commit
to SwissAS/esapi-java-legacy
that referenced
this issue
Apr 11, 2017
patcheng
pushed a commit
to patcheng/esapi-java-legacy
that referenced
this issue
May 24, 2017
xeno6696
pushed a commit
that referenced
this issue
Oct 9, 2018
* Close issue #448 * Close issue #444. Delete deprecated decodeToObject() method and 2 related encodeObject() methods. General whitespace clean-up. Delete EncoderTest.testBase64decodToObject() method which is no longer relevant. * Close issue #385. Close issue #386. NOTE: Was unwilling to comply with request in these 2 issues to make logSpecial() 'protected'. Could not do so without introducing potential security vulnerabilities. However, since the intent of the user submitting these 2 GitHub issues was only to override logSpecial() in order to completely suppress the output from them to stdout or stderr, I have arranged it so that setting the System property 'org.owasp.esapi.logSpecial.discard' to 'true' will do exactly this...it will suppress all output from logSpecial. (Also, the calls to System.err.println() and System.out.println() have been replaced by calls to logSpecial(), which will log to System.out if not suppressed. So the end result is all or nothing. I suspect that most will keep the default behavior, which is to print to System.out rather than suppressing all output.
Closed via PR #451 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
To avoid that the errors are writting to the console refactor the usage of System.err
The text was updated successfully, but these errors were encountered: