cxf-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Sergey Beryozkin (JIRA)" <>
Subject [jira] [Commented] (CXF-7309) JAX-RS @Context fields throw NPE in OSGI hot deployed filters
Date Fri, 26 May 2017 16:34:04 GMT


Sergey Beryozkin commented on CXF-7309:

well I applied a patch and did a commit to 3.1.x and then decided to revert, for now at least.
The reason is that I'm still honestly not 100% sure how safe it is, I found myself staring
at the code and realizing I was just not understanding the implications :-) 

Having to reflect every time the instance is injected into a tl proxy is a bit of a problem
(that check where you first try to extract from the getter) because that will affect all services,
with no hot redeployments haooeing around. 

I'm just thinking, in the production, would one really expect to redeploy a filter ?

I'm not saying this issue does not have to be addressed, I'm just having a bit of a doubt
at the moment. I thought after I saw my commit, that I need to first try to reproduce the
issue the original code was trying to address, and then do another iteration...
Thanks, Sergey

> JAX-RS @Context fields throw NPE in OSGI hot deployed filters
> -------------------------------------------------------------
>                 Key: CXF-7309
>                 URL:
>             Project: CXF
>          Issue Type: Bug
>          Components: JAX-RS
>    Affects Versions: 3.0.12, 3.1.10
>            Reporter: Ezequiel Rosas Garcia
>             Fix For: 3.2.0, 3.1.12
> Hello. 
> This happens with a PreMatching filter that is loaded from OSGI.
> I found that when the filter OSGI bundle is hot deployed, all other already running bundles
using it would start throwing NPE when trying to access the injected fields inside the filter
(like CXF-7248)
> This seems to happen due to other bundles retaining their ThreadLocal references in their
own AbstractResourceInfo#getSetterProxyMap() while the OSGI Proxy starts redirecting to a
new filter object which has new different ThreadLocal references as soon as it is used for
the first time after hot re deployment.
> Test: [Link](
> PR: [Link](

This message was sent by Atlassian JIRA

View raw message