Repository: bval Updated Branches: refs/heads/bv2 a63f3d5e0 -> 31a63ef13
ensure all exceptions thrown by ConstraintValidators are wrapped in ValidationExceptions Project: http://git-wip-us.apache.org/repos/asf/bval/repo Commit: http://git-wip-us.apache.org/repos/asf/bval/commit/31a63ef1 Tree: http://git-wip-us.apache.org/repos/asf/bval/tree/31a63ef1 Diff: http://git-wip-us.apache.org/repos/asf/bval/diff/31a63ef1 Branch: refs/heads/bv2 Commit: 31a63ef1303df93849cf8c32b142d630babea8e5 Parents: a63f3d5 Author: Matt Benson <mben...@apache.org> Authored: Tue Mar 6 18:03:47 2018 -0600 Committer: Matt Benson <mben...@apache.org> Committed: Tue Mar 6 18:03:47 2018 -0600 ---------------------------------------------------------------------- .../main/java/org/apache/bval/jsr/job/ValidationJob.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/bval/blob/31a63ef1/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java ---------------------------------------------------------------------- diff --git a/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java b/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java index 088f69d..87542f4 100644 --- a/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java +++ b/bval-jsr/src/main/java/org/apache/bval/jsr/job/ValidationJob.java @@ -135,8 +135,14 @@ public abstract class ValidationJob<T> { // null validator without exception implies composition: valid = true; } else { - constraintValidator.initialize(constraint.getAnnotation()); - valid = constraintValidator.isValid(context.getValue(), constraintValidatorContext); + try { + constraintValidator.initialize(constraint.getAnnotation()); + valid = constraintValidator.isValid(context.getValue(), constraintValidatorContext); + } catch (ValidationException e) { + throw e; + } catch (Exception e) { + throw new ValidationException(e); + } } if (!valid) { constraintValidatorContext.getRequiredViolations().forEach(sink);