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.