Skip to content

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

Closed
matthiaslarisch opened this issue Apr 11, 2017 · 1 comment
Closed

Avoid using System.err in EsapiPropertyManager #386

matthiaslarisch opened this issue Apr 11, 2017 · 1 comment
Assignees

Comments

@matthiaslarisch
Copy link

To avoid that the errors are writting to the console refactor the usage of System.err

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
@kwwall kwwall self-assigned this Oct 5, 2018
This was referenced Oct 5, 2018
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.
@kwwall
Copy link
Contributor

kwwall commented Dec 24, 2018

Closed via PR #451

@kwwall kwwall closed this as completed Dec 24, 2018
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