I've changed the exception list type from Throwable to Exception. It's more clear when @ExceptionHandler annotation use.
Issue: https://jira.spring.io/browse/SPR-11933
Comment From: milgarubanjuk
I have signed and agree to the terms of the SpringSource Individual Contributor License Agreement.
Comment From: rstoyanchev
@poutsma any thoughts? This is used primarily with Spring MVC where the HandlerExceptionResolver accepts Exception. Theoretically the annotation could be used elsewhere but perhaps it's worth aligning it more closely with Spring MVC.
Comment From: poutsma
@rstoyanchev Indeed, since the HandlerExceptionResolver only accepts Exception
, it makes sense to use that in @ExceptionResolver
as well. Unfortunately, that would break backward compatibility, I think.
Comment From: milgarubanjuk
hi~ any updates on this issue? Though that would break backward compatibility, that method only accepts Exception. I may think no one use throwable(or Error) type, so it doesn't cause any other issues.
Comment From: pivotal-issuemaster
@milgarubanjuk Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.
Comment From: rstoyanchev
The exception is now also used in WebFlux which does support Throwable.