Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Oct 21, 2014

...input) causes NPE if input is not valid.

Made changes as required, also getvalid(string,string) should be called instead.
Could not test the changes.
Let me know if the error persists.
Contact : [email protected]

…ng input) causes NPE if input is not valid.

Made changes as required, also getvalid(string,string) should be called instead.
Could not test the changes.
Let me know if the error persists.
Contact : [email protected]
@chrisisbeef
Copy link
Member

You should be able to use the existing tests as a guide for your test. The test should ensure that the bug is reproduced (test fails) before your fix is applied and after your fix is applied the test should pass.

karansanwal and others added 4 commits December 16, 2014 12:50
A minor mistake in previous commit
Issue 195 : Junit Tests.
Credits to quest313.
(Thanks!)
chrisisbeef added a commit that referenced this pull request Apr 27, 2015
Fix Issue # 195 : BaseValidationRule.assertValid(String context, String ...
@chrisisbeef chrisisbeef merged commit b4bd4e3 into ESAPI:master Apr 27, 2015
kwwall pushed a commit that referenced this pull request Dec 30, 2015
Merge of ESAPI/esapi-java-legacy into demkada/esapi-java-legacy
kwwall pushed a commit that referenced this pull request May 13, 2017
Update from ESAPI develop
xeno6696 pushed a commit to xeno6696/esapi-java-legacy that referenced this pull request Jun 27, 2025
Initial code to disable encodeForSQL
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

Successfully merging this pull request may close these issues.

2 participants