You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@struts.apache.org by "Lukasz Lenart (JIRA)" <ji...@apache.org> on 2013/04/18 09:17:15 UTC

[jira] [Commented] (WW-4050) AnnotationValidationInterceptor should consult UnknownHandler before throwing NoSuchMethodException

    [ https://issues.apache.org/jira/browse/WW-4050?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13634925#comment-13634925 ] 

Lukasz Lenart commented on WW-4050:
-----------------------------------

Could you prepare a small Maven based demo app?
                
> AnnotationValidationInterceptor should consult UnknownHandler before throwing NoSuchMethodException
> ---------------------------------------------------------------------------------------------------
>
>                 Key: WW-4050
>                 URL: https://issues.apache.org/jira/browse/WW-4050
>             Project: Struts 2
>          Issue Type: Improvement
>    Affects Versions: 2.3.14
>            Reporter: Azuo Lee
>             Fix For: 2.3.15
>
>
> If a {{NoSuchMethodException}} is caught when invoking {{getActionMethod()}}, {{AnnotationValidationInterceptor}} should first give any configured {{UnknownHandler}} a shot, before throwing the exception directly, just as what {{DefaultActionInvocation.invokeAction()}} does.
> Any unknown {{Action}} or unknown {{Result}} is currently handled by {{Dispatcher.serviceAction()}}, and a *404* error code will be returned as expected. But an unknown {{Method}} will raise a *500* error, the only way to avoid this is to configure an {{UnknownHandler}}. Therefore, {{AnnotationValidationInterceptor}} won't inspect {{UnknownHandler}} at all...
> {{GET /foo/list&nbsp;&nbsp;*404* (Action "foo" not found)}}
> {{GET /bar/list&nbsp;&nbsp;*200*}}
> {{GET /bar/print&nbsp;*500* (Method "print" not found, *404* is expected instead of *500*)}}

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira