[ 
https://issues.apache.org/jira/browse/WICKET-6290?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Carl-Eric Menzel updated WICKET-6290:
-------------------------------------
    Description: 
CssUrlReplacer tries to replace data: urls that are used to embed 
base64-encoded small images directly in CSS files. This leads to the following 
(from an adapted CssUrlReplacerTest I haven't pushed yet):

{code}
Expected :.class {background-image: 
url(data:image/gif;base64,R0lGODlhEAAQAMQAAORHH);}
Actual   :.class {background-image: 
url('./wicket/resource/org.apache.wicket.resource.CssUrlReplacerTest/res/css/data:image/gif;base64,R0lGODlhEAAQAMQAAORHH--decorated');}
{code}

It should really just leave those URLs alone. I'll prepare a pull request.

  was:
CssUrlReplacer tries to replace data: urls that are used to embed 
base64-encoded small images directly in CSS files. This leads to the following:

{code}
Expected :.class {background-image: 
url(data:image/gif;base64,R0lGODlhEAAQAMQAAORHH);}
Actual   :.class {background-image: 
url('./wicket/resource/org.apache.wicket.resource.CssUrlReplacerTest/res/css/data:image/gif;base64,R0lGODlhEAAQAMQAAORHH--decorated');}
{code}

It should really just leave those URLs alone. I'll prepare a pull request.


> CssUrlReplacer doesn't understand data: urls and breaks them
> ------------------------------------------------------------
>
>                 Key: WICKET-6290
>                 URL: https://issues.apache.org/jira/browse/WICKET-6290
>             Project: Wicket
>          Issue Type: Bug
>    Affects Versions: 8.0.0-M2, 6.25.0, 7.5.0, 1.5.17
>            Reporter: Carl-Eric Menzel
>            Assignee: Carl-Eric Menzel
>
> CssUrlReplacer tries to replace data: urls that are used to embed 
> base64-encoded small images directly in CSS files. This leads to the 
> following (from an adapted CssUrlReplacerTest I haven't pushed yet):
> {code}
> Expected :.class {background-image: 
> url(data:image/gif;base64,R0lGODlhEAAQAMQAAORHH);}
> Actual   :.class {background-image: 
> url('./wicket/resource/org.apache.wicket.resource.CssUrlReplacerTest/res/css/data:image/gif;base64,R0lGODlhEAAQAMQAAORHH--decorated');}
> {code}
> It should really just leave those URLs alone. I'll prepare a pull request.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to