fix: Maximum call stack size exceeded (#315)

* fix: Maximum call stack size exceeded

* test: increase timeout

* refactor: simplify test assertions by using expected variables

Author: D-Sketon

lib/renderer.js

@@ -225,6 +225,12 @@ module.exports = function(data, options) {
   const { prependRoot, postAsset, dompurify } = markedCfg;
   const { path, text } = data;
 
+  marked.defaults.extensions = null;
+  marked.defaults.tokenizer = null;
+  marked.defaults.renderer = null;
+  marked.defaults.hooks = null;
+  marked.defaults.walkTokens = null;
+
   // exec filter to extend marked
   this.execFilterSync('marked:use', marked.use, { context: this });
 

test/index.js

@@ -331,15 +331,21 @@ describe('Marked renderer', () => {
     ].join('\n');
 
     it('autolink enabled', () => {
-      const result = r({text: body});
-
-      result.should.eql([
+      let result = r({text: body});
+      const expected = [
         '<p>Great website <a href="http://hexo.io/">http://hexo.io</a></p>',
         '<p>A webpage <a href="http://www.example.com/">www.example.com</a></p>',
         '<p><a href="http://hexo.io/">Hexo</a></p>',
         '<p><a href="http://lorem.com/foo/">http://lorem.com/foo/</a></p>',
         '<p><a href="http://dolor.com/">http://dolor.com</a></p>'
-      ].join('\n') + '\n');
+      ].join('\n') + '\n';
+
+      result.should.eql(expected);
+
+      // try again
+      result = r({text: body});
+
+      result.should.eql(expected);
     });
 
     it('autolink disabled', () => {
@@ -354,6 +360,17 @@ describe('Marked renderer', () => {
         '<p><a href="http://dolor.com/">http://dolor.com</a></p>'
       ].join('\n') + '\n');
     });
+
+    it('should not stack overflow', function() {
+      this.timeout(5000);
+      const body = 'Great website http://hexo.io';
+
+      (() => {
+        for (let i = 0; i < 100000; i++) {
+          r({text: body});
+        }
+      }).should.not.throw();
+    });
   });
 
   describe('mangle', () => {
@@ -924,12 +941,17 @@ describe('Marked renderer', () => {
 
       const r = require('../lib/renderer').bind(hexo);
 
-      const result = r({text: body});
-
-      result.should.eql([
+      let result = r({text: body});
+      const expected = [
         `<p><img data-src="${encodeURL(urlA)}">`,
         `<img data-src="${encodeURL(urlB)}"></p>\n`
-      ].join('\n'));
+      ].join('\n');
+
+      result.should.eql(expected);
+      // try again
+      result = r({text: body});
+
+      result.should.eql(expected);
     });
 
     it('should execute filter registered to marked:tokenizer', () => {
@@ -951,8 +973,12 @@ describe('Marked renderer', () => {
 
       const r = require('../lib/renderer').bind(hexo);
 
-      const result = r({text: body});
-      result.should.eql(`<p>${escapeHTML(smartypants(body))}</p>\n`);
+      let result = r({text: body});
+      const expected = `<p>${escapeHTML(smartypants(body))}</p>\n`;
+      result.should.eql(expected);
+      // try again
+      result = r({text: body});
+      result.should.eql(expected);
     });
 
     it('should execute filter registered to marked:extensions', () => {
@@ -983,8 +1009,12 @@ describe('Marked renderer', () => {
 
       const r = require('../lib/renderer').bind(hexo);
 
-      const result = r({text: body});
-      result.should.eql(`<p class="math block">${escapeHTML('E=mc^2')}</p>\n`);
+      let result = r({text: body});
+      const expected = `<p class="math block">${escapeHTML('E=mc^2')}</p>\n`;
+      result.should.eql(expected);
+      // try again
+      result = r({text: body});
+      result.should.eql(expected);
     });
   });