natas_hw
/
dubbo
0
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

issue5432: avoid of serialization exception for javax.validation.ConstraintViolationException (#5672)

This commit is contained in:
Ian Luo 2020-01-17 14:34:56 +08:00 committed by ken.lj
parent a9a69eac67
commit 0f99cb65c3
1 changed files with 5 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
dubbo-filter/dubbo-filter-validation/src/main/java/org/apache/dubbo/validation/filter/ValidationFilter.java
View File

@ -27,6 +27,8 @@ import org.apache.dubbo.rpc.RpcException;
import org.apache.dubbo.validation.Validation; import org.apache.dubbo.validation.Validation;
import org.apache.dubbo.validation.Validator; import org.apache.dubbo.validation.Validator;
import javax.validation.ValidationException;
import static org.apache.dubbo.common.constants.CommonConstants.CONSUMER; import static org.apache.dubbo.common.constants.CommonConstants.CONSUMER;
import static org.apache.dubbo.common.constants.CommonConstants.PROVIDER; import static org.apache.dubbo.common.constants.CommonConstants.PROVIDER;
import static org.apache.dubbo.common.constants.FilterConstants.VALIDATION_KEY; import static org.apache.dubbo.common.constants.FilterConstants.VALIDATION_KEY;
@ -89,6 +91,9 @@ public class ValidationFilter implements Filter {
} }
} catch (RpcException e) { } catch (RpcException e) {
throw e; throw e;
} catch (ValidationException e) {
// only use exception's message to avoid potential serialization issue
return AsyncRpcResult.newDefaultAsyncResult(new ValidationException(e.getMessage()), invocation);
} catch (Throwable t) { } catch (Throwable t) {
return AsyncRpcResult.newDefaultAsyncResult(t, invocation); return AsyncRpcResult.newDefaultAsyncResult(t, invocation);
} }