[
https://issues.apache.org/jira/browse/SHINDIG-1411?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12900974#action_12900974
]
Gagandeep Singh commented on SHINDIG-1411:
------------------------------------------
Codereview uploaded : http://codereview.appspot.com/2004042/
> CSS rewriter doesnt take container into account when rewriting embedded URL's
> -----------------------------------------------------------------------------
>
> Key: SHINDIG-1411
> URL: https://issues.apache.org/jira/browse/SHINDIG-1411
> Project: Shindig
> Issue Type: Bug
> Components: Java
> Affects Versions: 2.0.0-RC2
> Reporter: Jesse Ciancetta
> Attachments: CSSRewriterPatch.patch
>
>
> I ran into a container configuration related issue while testing out RC2,
> which is that the CSS rewriter doesn't seem to be taking the container into
> account when rewriting embedded URL's -- it always uses the default container
> configuration which ends up giving me invalid proxy URL's in my rewritten
> CSS. This appears to be due to the fact that the CssResponseRewriter uses
> the DomWalker.makeGadget method to create a Gadget instance to pass to the
> ProxyUriManager.ProxyUri constructor, but the GadgetContext inside that
> Gadget instance doesnt have the container set, which makes code that runs
> later in the ProxyUriManager to generate the proxy URL use the default
> container configuration instead of my custom container configuration.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
