I've written a content management system that uses a server-side regular expression to escape ampersands in the page response just prior to it being sent to the client's browser. The regular expression is mindful of ampersands that have already been escaped or are part of an HTML entity. For example, the following:
a & b, c & d, © 2009
gets changed to this:
a & b, c & d, © 2009
(Only the first & is modified.) Here is the regular expression, which was taken and modified from a Rails helper:
html.gsub(/&(?!([a-zA-Z][a-zA-Z0-9]*|(#\d+));)/) { |special| ERB::Util::HTML_ESCAPE[special] }
While this works great, it does have a problem. The regular expression is not aware of any <![CDATA[ or ]]> that might be surrounding the unescaped ampersands. This is necessary for embedded JavaScript to remain untouched. For example, this:
<script type="text/javascript">
// <![CDATA[
if (a && b) doSomething();
// ]]>
</script>
is unfortunately rendered as this:
<script type="text/javascript">
// <![CDATA[
if (a && b) doSomething();
// ]]>
</script>
which of course the JavaScript engines don't understand.
My question is this: Is there a way to modify the regular expression to do exactly as it is doing now with the exception that it leaves text inside a CDATA section untouched?
Since the regular expression is not so simple to begin with, this question might be easier to answer: Is it possible to write a regular expression that will change all letters into a period except those letters between a '<' and a '>'? For example, one that would change "some <words> are < safe! >" into ".... <words> ... < safe! >"?
-
I've done something similar here:
http://stackoverflow.com/questions/157646/best-way-to-encode-text-data-for-xmlFortunately, in my case CDATA wasn't a problem.
What is a problem is that you have to be careful that the expression isn't greedy or you'll end up with something like this:
.... <words> are < safe! > -
I seriously doubt that what you are trying to accomplish is something you can do using a regular expression alone. Regexps are notoriously bad at correctly handing nesting.
You will probably be better off using an XML parser and not escaping CDATA content.
-
Don't use regular expressions for this. It is a terrible, terrible idea. Instead, simply HTML encode anything that you're outputting that might have a character in it. Like this:
require 'cgi' print CGI.escape("All of this is HTML encoded!")Ben Blank : Wouldn't this cause already-escaped entities to become doubly-encoded? (e.g. `&` -> `&`?)Nick : I don't want to escape everything for a few reasons, one being that (as Ben Blank said) & would become & but also because I don't want characters in inline JavaScript to be escaped, hence the need to exclude CDATA sections.Evan Fosmark : Whoops. I should have said unescape instead. -
You asked for it! :D
/&(?!(?:[a-zA-Z][a-zA-Z0-9]*|#\d+);) (?!(?>(?:(?!<!\[CDATA\[|\]\]>).)*)\]\]>)/xmThe first line is your original regex. The lookahead matches if there's a CDATA closing sequence (
]]>) up ahead, unless there's an opening sequence (<!CDATA[) between here and there. Assuming the document is minimally well formed, that should mean the current position is inside a CDATA section.Oops, I had that backward: by using positive lookahead I was matching "naked" ampersands only within CDATA sections. I changed it to a negative lookahead, so now it works right.
By the way, this regex works in RegexBuddy in Ruby mode, but not at the rubular site. I suspect Rubular uses an older version of Ruby with less-powerful regex support; can anyone confirm that? (As you may have guessed, I'm not a Ruby programmer.)
EDIT: The problem at Rubular was that I used 's' as a modifier (to mean dot-matches-everything), but Ruby uses 'm' for that.
Chase Seibert : Nice solution. This took me quite a while to grok. Here is a detailed explanation if anyone else is interested: http://bitkickers.blogspot.com/2009/01/regular-expression-negative-lookahead_31.htmlAlan Moore : "I think this is self-explanatory. See you next time!" :D -
That worked! At Rubular I had to change the options from
/xsto/m(and I removed the whitespace that separates the two parts of the regex as you showed it above).You can see this regular expression in action along with a sample string at http://www.rubular.com/regexes/5855.
In case that Rubular permalink isn't really permanent, here is what I entered for the regular expression:
/&(?!(?:[a-zA-Z][a-zA-Z0-9]*|#\d+);)(?!(?>(?:(?!<!\[CDATA\[|\]\]>).)*)\]\]>)/mAnd here is the test string:
<p>a & b</p> <p>c & d</p> <script type="text/javascript"> // <![CDATA[ if (a && b) doSomething('a & b & c'); // ]]> </script> <p>a & b</p> <p>c & d</p>Only two ampersands match -- the
a & bat the top and thea & bat the bottom. Ampersands already escaped as&and all ampersands (escaped or not) between<![CDATA[and]]>are left alone.So, my final code is now this:
html.gsub(/&(?!(?:[a-zA-Z][a-zA-Z0-9]*|#\d+);)(?!(?>(?:(?!<!\[CDATA\[|\]\]>).)*)\]\]>)/m, '&')Thank you very much Alan. This is exactly what I needed.
Alan Moore : Ach! I keep forgetting about Ruby using the 'm' modifier to mean what everybody else uses 's' for. I'll fix that.
0 comments:
Post a Comment