You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@cxf.apache.org by "Ravi Luthra (JIRA)" <ji...@apache.org> on 2015/06/30 21:01:04 UTC

[jira] [Updated] (CXF-6480) InjectionUtils createThreadLocalServletApiContext better error message for invalid @Context classes

     [ https://issues.apache.org/jira/browse/CXF-6480?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Ravi Luthra updated CXF-6480:
-----------------------------
    Priority: Trivial  (was: Major)

> InjectionUtils createThreadLocalServletApiContext better error message for invalid @Context classes
> ---------------------------------------------------------------------------------------------------
>
>                 Key: CXF-6480
>                 URL: https://issues.apache.org/jira/browse/CXF-6480
>             Project: CXF
>          Issue Type: Bug
>         Environment: All OS
>            Reporter: Ravi Luthra
>            Priority: Trivial
>
> When you have an \@Context field in a JAXRS class (maybe JAXWS too?) there is a select list of possible proxy classes. This method checks for the proxy class using an if/else/if/else... structure, but there is no final else clause with a good exception. Instead the method attempts to instantiate a null class name (proxyClassName), which throws a NullPointerException. This leaves the method wrapped inside a RuntimeException. A common forgetful change someone might make is to add
> {code}@Context private HttpSession session; //(instead of private HttpServletRequest){code}
> Which leaves a massive stack trace leaving just an NPE and Spring also tends to obscure the stack-trace of this issue. A better error would be to suggest that the given class ...HttpSession is not a valid Context type. No proxy classes can be created for this type.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)