BTW, the sleep doesn't seem to affect this one way or the other.

On Tue, Apr 15, 2014 at 11:36 AM, Nils Kilden-Pedersen <[email protected]>wrote:

> Sorry about the delay.
>
> I tried this on 9.1.4 and got the same problem.
>
> This servlet:
>
> @WebServlet(asyncSupported=true, urlPatterns={"/hello"})public class 
> MyServlet extends HttpServlet {
>
>     private static final Executor EXEC = Executors.newCachedThreadPool();
>
>     public void doGet(final HttpServletRequest req, final HttpServletResponse 
> res) {
>         final AsyncContext async = req.startAsync();
>         EXEC.execute(new Runnable() {
>             public void run() {
>                 try {
>                     res.setContentType("text/plain");
>                     Thread.sleep(2000);
>                     PrintWriter out = res.getWriter();
>                     out.println("Context path: " + req.getContextPath());
>                     out.println("Servlet path: " + req.getServletPath());
>                 } catch (Exception e) {
>                     e.printStackTrace();
>                 } finally {
>                     async.complete();
>                 }
>             }
>         });
>     }
> }
>
> when put in webapps/ROOT produces this:
>
> Context path: null
> Servlet path:
>
>
>
> On Sun, Apr 6, 2014 at 8:06 PM, Jan Bartel <[email protected]> wrote:
>
>> Hi Nils,
>>
>> Can you make a really simple test that reproduces?
>>
>> regards
>> Jan
>>
>> On 6 April 2014 04:18, Nils Kilden-Pedersen <[email protected]> wrote:
>> > I suspect this is a race condition related to async. If the Request
>> object
>> > (or by extension HttpConnection), it looks like the context path (and
>> other
>> > mutable variables) are being set/reset to null, possibly before the
>> async
>> > context has completed.
>> >
>> > When I disabled async I could not reproduce this.
>> >
>> > On Sat, Apr 5, 2014 at 9:00 AM, Nils Kilden-Pedersen <[email protected]>
>> > wrote:
>> >>
>> >> Here are some characteristics of my app:
>> >>
>> >> Jetty 9.1.2
>> >>
>> >> Windows 8.1, haven't yet tried on Linux (probably not relevant)
>> >>
>> >> App is a folder in webapps named ROOT
>> >> There's no xml descriptor
>> >> The servlet is configured using annotations
>> >> The servlet is async
>> >>
>> >> So very basic setup, configuration wise.
>> >>
>> >>
>> >>
>> >> On Sat, Apr 5, 2014 at 3:27 AM, Jan Bartel <[email protected]> wrote:
>> >>>
>> >>> Nils,
>> >>>
>> >>> Using the standard test.war and test.xml from the jetty distro, and
>> >>> setting the context path to be "/", I haven't been able to reproduce
>> >>> this with jetty-9.1.3, nor actually with jetty-9.1.0. Can you reliably
>> >>> reproduce with the test webapp?
>> >>>
>> >>> Jan
>> >>>
>> >>> On 5 April 2014 14:05, Nils Kilden-Pedersen <[email protected]> wrote:
>> >>> > When running in the root context, request.getContextPath() returns
>> null
>> >>> > rather than "" as expected.
>> >>> >
>> >>> >
>> >>> >
>> http://docs.oracle.com/javaee/6/api/javax/servlet/http/HttpServletRequest.html#getContextPath()
>> >>> >
>> >>> > This is on 9.1.2.
>> >>> >
>> >>> >
>> >>> > _______________________________________________
>> >>> > jetty-users mailing list
>> >>> > [email protected]
>> >>> > https://dev.eclipse.org/mailman/listinfo/jetty-users
>> >>> >
>> >>>
>> >>>
>> >>>
>> >>> --
>> >>> Jan Bartel <[email protected]>
>> >>> www.webtide.com
>> >>> 'Expert Jetty/CometD developer,production,operations advice'
>> >>> _______________________________________________
>> >>> jetty-users mailing list
>> >>> [email protected]
>> >>> https://dev.eclipse.org/mailman/listinfo/jetty-users
>> >>
>> >>
>> >
>> >
>> > _______________________________________________
>> > jetty-users mailing list
>> > [email protected]
>> > https://dev.eclipse.org/mailman/listinfo/jetty-users
>> >
>>
>>
>>
>> --
>> Jan Bartel <[email protected]>
>> www.webtide.com
>> 'Expert Jetty/CometD developer,production,operations advice'
>> _______________________________________________
>> jetty-users mailing list
>> [email protected]
>> https://dev.eclipse.org/mailman/listinfo/jetty-users
>>
>
>
_______________________________________________
jetty-users mailing list
[email protected]
https://dev.eclipse.org/mailman/listinfo/jetty-users

Reply via email to