Raise descriptive exception in case parameter array passed to validation engine has wrong length

Activity

Show:

Hardy FerentschikOctober 9, 2013 at 3:05 PM

Right, I've added a parameter length check here - https://github.com/hferentschik/hibernate-validator/tree/HV-805, but as expected the TCK tests are failing now. +1 for doing a TCK release. IMO sooner than later.

Gunnar MorlingOctober 9, 2013 at 2:12 PM

Yes, needs to be resolved before. We could do a joint release of TCK 1.1.2 and HV with a fix for HV-805.

Hardy FerentschikOctober 9, 2013 at 2:09 PM

Except we cannot fail early w/o a TCK update.

Gunnar MorlingOctober 9, 2013 at 2:06 PM

It seems not be be working in all cases. When e.g. changing GarageTest.java#L33 into an empty array, an ArrayIndexOutOfBoundsException is thrown. I think safest would be to perform a length check quite early in the call flow.

Hardy FerentschikOctober 9, 2013 at 1:46 PM
Edited

I think is already addressed by HV-672. There is also WrongMethodOrParameterForMethodValidationTest.

Fixed

Details

Assignee

Reporter

Participants

Gunnar Morling
Hardy Ferentschik

Components

Fix versions

Affects versions

Priority

Created July 23, 2013 at 1:34 PM
Updated January 28, 2014 at 9:51 AM
Resolved December 2, 2013 at 4:52 PM