[ 
https://issues.apache.org/jira/browse/TRINIDAD-1233?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12794520#action_12794520
 ] 

Blake Sullivan commented on TRINIDAD-1233:
------------------------------------------

If the encoding only needs to be done for Porlets, then that is the only case 
in which is should be done.  We shouldn't be encoding for Servlets when we 
explicitly know that we don't want it and then ripping out the results.  It 
isn't clear that we would need any of the jsessionId removal code in the 
portlet case.  Note that I also suspect that even in the Portlet case, encoding 
the resource URL will add more context than we really want.  We really want all 
of the same skin for all of the portlets served by the same portlet provider to 
have the same URL.

In short, I think that this patch is exactly the wrong way to solve the 
underlying issue

> URLs in generated CSS might include ;jessionid=xxxx
> ---------------------------------------------------
>
>                 Key: TRINIDAD-1233
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-1233
>             Project: MyFaces Trinidad
>          Issue Type: Improvement
>    Affects Versions: 1.2.9-core
>         Environment: Trinidad 1.2.10-SNAPSHOT
> tomcat 6.0.14
> tomcat 5.5
>            Reporter: Harald Kuhn
>         Attachments: CSSUtils.java.patch
>
>
> Depending on the Servlet-Container URLs in the generated CSS-file may include 
> the Session-Id (of the session that initiated the generation). 
> i.e.
> background-image:url(/aim/css/skin/airplus/img/dialogclose.gif;jsessionid=8A002E216A7B2BF42735740787DBF102.tomcat)
> This part of the URL should be removed.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.

Reply via email to