Re: [jetty-users] WAR overlays in Jetty 9.1?

[Jan Bartel <janb@xxxxxxxxxxx>]

Regarding bug (where I've added more comments):  https://bugs.eclipse.org/423361

I think the problem was fixed for 9.1 by this checkin:
https://github.com/eclipse/jetty.project/commit/73bee3af4bc951344ece3004671f4169bf68ee9f

At least, with that checkin, I ensured that listeners from web.xml and
programmatic listeners would be subject to jetty's Decorator mechanism
that I believe Weld has implemented to make them CDI injectable.

If anyone has an app that can test that out to confirm that listener
CDI injection works with jetty-9.1, that would be great. When its
confirmed as working, I will edit the jetty doco page here:
http://www.eclipse.org/jetty/documentation/current/framework-weld.html
to remove the caveat about ServletContextListeners.

BTW, I've made a change to ensure that listeners that are @WebListener
annotations will also be injectable - that fix will go out in the next
9.1.1 release. I don't think that will be the cause of any issues for
Weld, just something I noticed on code inspection.

cheers
Jan

On 6 December 2013 06:40, Joakim Erdfelt <joakim@xxxxxxxxxxx> wrote:
> Filed a few more Jetty side issues around CDI ...
>
> CDI improvement issues filed.
>
> Servlet
> https://bugs.eclipse.org/423361 - CDI / Servlet Listeners should be created
> via ServletContextHandler.createInstance(Class)
>
> java.websocket (JSR-356)
> https://bugs.eclipse.org/423336 - CDI / WebSocket / JSR / New Endpoints
> should be created via ServletContextHandler.createInstance(Class)
> https://bugs.eclipse.org/423363 - CDI / WebSocket / JSR / New Decoders
> should be created via ServletContextHandler.createInstance(Class)
> https://bugs.eclipse.org/423364 - CDI / WebSocket / JSR / New Encoders
> should be created via ServletContextHandler.createInstance(Class)
> https://bugs.eclipse.org/423365 - CDI / WebSocket / JSR / New
> ServerEndpointConfig.Configurators should be created via
> ServletContextHandler.createInstance(Class)
> https://bugs.eclipse.org/423367 - CDI / WebSocket / JSR / New
> ServerApplicationConfig objects should be created via
> ServletContextHandler.createInstance(Class)
>
>
>
> --
> Joakim Erdfelt <joakim@xxxxxxxxxxx>
> webtide.com - intalio.com/jetty
> Expert advice, services and support from from the Jetty & CometD experts
> eclipse.org/jetty - cometd.org
>
>
> On Thu, Dec 5, 2013 at 11:30 AM, Klaus Brunner <klaus.brunner@xxxxxxxxx>
> wrote:
>>
>> 2013/12/5 Joakim Erdfelt <joakim@xxxxxxxxxxx>:
>> > Can you expand on this ...
>> >
>> > "Except of course for the longer-standing problem of no
>> > injection in Listeners...it would be fantastic if that could be done."
>> >
>> > Is there a WELD or Jetty bug filed about this?
>>
>> Well, sort of. There's this in the documentation at
>> http://www.eclipse.org/jetty/documentation/current/framework-weld.html
>> :
>>
>> "At the moment, only filters and servlets are supported for CDI with
>> Weld. ServletContextListeners can not be configured with CDI (until we
>> sort out the proper solution to the issue)."
>>
>>
>> Klaus
>> _______________________________________________
>> jetty-users mailing list
>> jetty-users@xxxxxxxxxxx
>> https://dev.eclipse.org/mailman/listinfo/jetty-users
>
>
>
> _______________________________________________
> jetty-users mailing list
> jetty-users@xxxxxxxxxxx
> https://dev.eclipse.org/mailman/listinfo/jetty-users
>



-- 
Jan Bartel <janb@xxxxxxxxxxx>
www.webtide.com
'Expert Jetty/CometD developer,production,operations advice'