You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by "Carl-Eric Menzel (JIRA)" <ji...@apache.org> on 2016/12/05 10:52:58 UTC
[jira] [Closed] (WICKET-6290) CssUrlReplacer doesn't understand
data: urls and breaks them
[ https://issues.apache.org/jira/browse/WICKET-6290?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Carl-Eric Menzel closed WICKET-6290.
------------------------------------
Resolution: Fixed
Fix Version/s: 8.0.0-M3
7.6.0
6.26
Pull requests merged for 6, 7 and 8.
> 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
> Fix For: 6.26, 7.6.0, 8.0.0-M3
>
>
> 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)