https://bugs.eclipse.org/bugs/show_bug.cgi?id=433321


On Thu, Apr 17, 2014 at 8:49 AM, Nils Kilden-Pedersen <[email protected]>wrote:

> Should I open a bug report?
>
>
> 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