Skip to content

Fix To Form Auth Adapter Validation With Field Mapping #440

Merged
merged 1 commit into from Apr 30, 2012

3 participants

@rmarscher

Fixes #439. Found that the existing testValidatorWithFieldMapping test wasn't making sure that the password wasn't sent in the query which is why the tests were passing previously. It now maps the validator keys according to the field mapping to correctly remove the data used in post-query validation from the query.

@gwoo
Union of RAD member
gwoo commented Apr 26, 2012

Seems that the documentation for this may be out of date. Could you double check?

@rmarscher

Hey @gwoo, as far as I can tell the documentation is correct - and this now works properly according to the documentation. I think no one was using the password mapping in real-life and didn't catch the problem because of the issue in the unit test.

@nateabele nateabele merged commit 89d4ad5 into UnionOfRAD:dev Apr 30, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.