Escape parenthesis in link URLs. Fix #459.

mixmark-io · Apr 3, 2024 · ebefa8c · ebefa8c
1 parent 89ef473
commit ebefa8c
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 0 deletions.
diff --git a/src/commonmark-rules.js b/src/commonmark-rules.js
@@ -153,6 +153,7 @@ rules.inlineLink = {
 
   replacement: function (content, node) {
     var href = node.getAttribute('href')
+    if (href) href = href.replace(/([()])/g, '\\$1')
     var title = cleanAttribute(node.getAttribute('title'))
     if (title) title = ' "' + title + '"'
     return '[' + content + '](' + href + title + ')'

diff --git a/test/index.html b/test/index.html
@@ -217,6 +217,11 @@
 link")</pre>
 </div>
 
+<div class="case" data-name="a with parenthesis in a query">
+  <div class="input"><a href="http://example.com?(query)">An anchor</a></div>
+  <pre class="expected">[An anchor](http://example.com?\(query\))</pre>
+</div>
+
 <div class="case" data-name="a without a src">
   <div class="input"><a id="about-anchor">Anchor without a title</a></div>
   <pre class="expected">Anchor without a title</pre>