Escape html in tag and bump markdown-it

svbergerem · Jan 11, 2015 · a5c59ee · a5c59ee
1 parent 1cc9321
commit a5c59ee
Show file tree

Hide file tree

Showing 5 changed files with 19 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,9 @@
+# 0.2.2
+
+* Move the class attribute to the end
+* Escape html in the tag
+* Bump markdown-it
+
 # 0.2.1
 
 * Update files in /dist

diff --git a/index.js b/index.js
@@ -24,6 +24,7 @@ function isLinkClose(str) { return /^<\/a\s*>/i.test(str); }
 module.exports = function hashtag_plugin(md, options) {
 
   var arrayReplaceAt = md.utils.arrayReplaceAt;
+  var escapeHtml = md.utils.escapeHtml;
 
   function hashtag(state) {
     var i, j, l, m,
@@ -110,7 +111,7 @@ module.exports = function hashtag_plugin(md, options) {
           });
           nodes.push({
             type: 'hashtag_text',
-            content: tagName,
+            content: escapeHtml(tagName),
             level: level
           });
           nodes.push({

diff --git a/package.json b/package.json
@@ -29,7 +29,7 @@
     "eslint-plugin-nodeca": "^1.0.0",
     "istanbul": "*",
     "lodash": "*",
-    "markdown-it": "~3.0.1",
+    "markdown-it": "~3.0.3",
     "markdown-it-testgen": "~0.1.0",
     "mocha": "*",
     "request": "*",

diff --git a/test/fixtures/hashtag/options.txt b/test/fixtures/hashtag/options.txt
@@ -79,13 +79,13 @@ missing preceeding#space
 .
 #<3
 .
-<p><a href="/tags/<3" class="tag">#<3</a></p>
+<p><a href="/tags/<3" class="tag">#&lt;3</a></p>
 .
 
 .
 #<3 and other #hashtags
 .
-<p><a href="/tags/<3" class="tag">#<3</a> and other <a href="/tags/hashtags" class="tag">#hashtags</a></p>
+<p><a href="/tags/<3" class="tag">#&lt;3</a> and other <a href="/tags/hashtags" class="tag">#hashtags</a></p>
 .
 
 .

diff --git a/test/fixtures/markdown-it/commonmark_extras.txt b/test/fixtures/markdown-it/commonmark_extras.txt
@@ -52,6 +52,14 @@ Issue #55:
 .
 
 
+Issue #35. `<` shoud work as punctuation
+.
+an **(:**<br>
+.
+<p>an <strong>(:</strong><br></p>
+.
+
+
 Should unescape only needed things in link destinations/titles:
 .
 [test](<\f\o\o\>\\>)