User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.6.2
Thanks, I will do PR, because I already have this commit :-)
Petr
On 3/22/22 15:30, Kyle Aure wrote:
Hello Arjan,
Thanks for pointing this out. As I was porting these tests
from the Platform TCK to a Standalone TCK many of the
@WebServlet URL Paths were declared as constants in separate
Constant.java files.
In order to make the TCK more readable, I wanted to remove
those constants and just use simple strings for the servlet
URL.
I think sometime during that process the leading slash was
removed and I didn't notice it because Liberty handles that
case (the platform I was using to test).
I can go in and add the leading slash back in if it is
necessary per spec, or if it is helpful for other
implementors.
Thanks, so I guess Kyle would be the first one to ask
about this. I'm cc'ing him here.
I wonder if it might be related to the injected
Arquillian URL ending or not ending on a /. I've seen
issues with that before. Maybe Kyle made this change to
fix the Arquillian connector on Liberty.
I'm looking at the TCK tests for Concurrent
and one of the interesting things is removing
leading slash in the @WebServlet annotation.
Even the existing ones were removed.