[weld-issues] [JBoss JIRA] Commented: (CDITCK-191) double slash in Conversation test urls break redirect

[Pete Muir (JIRA)]

    [ https://issues.jboss.org/browse/CDITCK-191?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12570283#comment-12570283 ] 

Pete Muir commented on CDITCK-191:
----------------------------------

Ok, comments updated, thanks for the feedback! As you guessed, we run this very much as an OSS project, and so appreciate the communities involvement!

I've removed that test in the branch, that was a mistake on my part! I will also fix Gurkan's issue and do a new release later today.

> double slash in Conversation test urls break redirect
> -----------------------------------------------------
>
>                 Key: CDITCK-191
>                 URL: https://issues.jboss.org/browse/CDITCK-191
>             Project: CDI TCK
>          Issue Type: Bug
>      Security Level: Public(Everyone can see) 
>          Components: Tests
>    Affects Versions: 1.0.2.CR3
>            Reporter: David Jencks
>            Assignee: Pete Muir
>             Fix For: 1.0.3.Final, 1.1.0.CR1
>
>         Attachments: CDITCK-191.diff
>
>
> The Conversation tests all call AbstractConversationTest.getPath() with strings starting with / such as /storm.jspx.  Since the method tacks a / at the end of the context root you end up with urls for the form foo//bar.jsf.  While tomcat appears to think that this is the same url as foo/bar.jsf, jetty doesn't.  This breaks the redirect in LongRunningConversationPropagatedByFacesContextTest.testConversationPropagatedOverRedirect.
> Removing the / from the arguments to getPath fixes the problem.

-- 
This message is automatically generated by JIRA.
-
For more information on JIRA, see: http://www.atlassian.com/software/jira