Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Site with html in CDATA tag in script tag gets broken by rewrite_javascript #542

Closed
GoogleCodeExporter opened this issue Apr 6, 2015 · 3 comments

Comments

@GoogleCodeExporter
Copy link

http://www.pro-it-support.co.uk/?ModPagespeed=off has this:

<script type="text/javascript">
<![CDATA[<link 
href='http://fonts.googleapis.com/css?family=Droid+Sans:400,700&subset=latin,lat
in-ext' rel='stylesheet' type='text/css'>]]>
</script>

http://www.pro-it-support.co.uk/?ModPagespeedFilters=rewrite_javascript has 
this:

<script type="text/javascript">//<![CDATA[
<![CDATA[<link 
href='http://fonts.googleapis.com/css?family=Droid+Sans:400,700&subset=latin,lat
in-ext'rel='stylesheet'type='text/css'>]]>
//]]></script>

I think mod_pagespeed should not have added the extra CDATA tag.

Original issue reported on code.google.com by jmara...@google.com on 18 Oct 2012 at 12:12

@GoogleCodeExporter
Copy link
Author

Original comment by jmara...@google.com on 1 Nov 2012 at 3:33

  • Changed state: Started

@GoogleCodeExporter
Copy link
Author

This issue was closed by revision r2125.

Original comment by jmara...@google.com on 1 Nov 2012 at 5:56

  • Changed state: Fixed

@GoogleCodeExporter
Copy link
Author

Original comment by jmara...@google.com on 1 Nov 2012 at 5:56

  • Added labels: Milestone-v23, release-note

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant