https://gist.github.com/jnorthrup/7bd8530546c55db30990d79302722d0c/edit
shows before and after coroutine removal

On Fri, Feb 15, 2019 at 2:59 PM Simone Bordet <[email protected]> wrote:

> Hi,
>
>
> On Fri, Feb 15, 2019 at 6:29 AM James Northrup <[email protected]> wrote:
> >
> > revising the servletcontextholder changes the result:
> https://gist.github.com/jnorthrup/792a43d73178ee38d686035fca3d146d
>
> It NPEs at PushCacheFilter.java:114, which is:
>
> config.getServletContext().setAttribute(config.getFilterName(), this);
>
> A NPE there is not possible, unless "config" is null, or the
> ServletContext is null.
> In both cases it is a major break of the implementation of the Servlet
> specification, which I doubt it's the case (we would have a million
> reports by now).
>
> Have you tried to _not_ use coroutines, at least to initialize Jetty?
>
> I would recommend that you start with the simplest possible Jetty
> server configuration: a server, one connector, one
> ServletContextHandler.
> Try that and see if it works. Then start adding things until it breaks.
>
> Since it's Kotlin, try also to write it in pure Java first and then
> convert to Kotlin.
>
> --
> Simone Bordet
> ----
> http://cometd.org
> http://webtide.com
> Developer advice, training, services and support
> from the Jetty & CometD experts.
> _______________________________________________
> jetty-users mailing list
> [email protected]
> To change your delivery options, retrieve your password, or unsubscribe
> from this list, visit
> https://www.eclipse.org/mailman/listinfo/jetty-users
>
_______________________________________________
jetty-users mailing list
[email protected]
To change your delivery options, retrieve your password, or unsubscribe from 
this list, visit
https://www.eclipse.org/mailman/listinfo/jetty-users

Reply via email to