diff --git a/src/Lexer.js b/src/Lexer.js --- a/src/Lexer.js +++ b/src/Lexer.js @@ -6,5 +6,5 @@ import { repeatString } from './helpers.js'; /** * smartypants text replacement - */ + * function smartypants(text) { return text @@ -27,5 +27,5 @@ function smartypants(text) { /** * mangle email addresses - */ + * function mangle(text) { let out = '', @@ -466,5 +466,5 @@ export class Lexer { // autolink - if (token = this.tokenizer.autolink(src, mangle)) { + if (token = this.tokenizer.autolink(src)) { src = src.substring(token.raw.length); tokens.push(token); @@ -473,5 +473,5 @@ export class Lexer { // url (gfm) - if (!this.state.inLink && (token = this.tokenizer.url(src, mangle))) { + if (!this.state.inLink && (token = this.tokenizer.url(src))) { src = src.substring(token.raw.length); tokens.push(token); @@ -494,5 +494,5 @@ export class Lexer { } } - if (token = this.tokenizer.inlineText(cutSrc, smartypants)) { + if (token = this.tokenizer.inlineText(cutSrc)) { src = src.substring(token.raw.length); this.ln = token.ln || this.ln; diff --git a/src/Renderer.js b/src/Renderer.js --- a/src/Renderer.js +++ b/src/Renderer.js @@ -142,5 +142,5 @@ export class Renderer { link(href, title, text) { - href = cleanUrl(this.options.sanitize, this.options.baseUrl, href); + href = cleanUrl(this.options.baseUrl, href); if (href === null) { return text; @@ -155,5 +155,5 @@ export class Renderer { image(href, title, text) { - href = cleanUrl(this.options.sanitize, this.options.baseUrl, href); + href = cleanUrl(this.options.baseUrl, href); if (href === null) { return text; diff --git a/src/Tokenizer.js b/src/Tokenizer.js --- a/src/Tokenizer.js +++ b/src/Tokenizer.js @@ -320,14 +320,7 @@ export class Tokenizer { type: 'html', raw: cap[0], - pre: !this.options.sanitizer - && (cap[1] === 'pre' || cap[1] === 'script' || cap[1] === 'style'), + pre: (cap[1] === 'pre' || cap[1] === 'script' || cap[1] === 'style'), text: cap[0] }; - if (this.options.sanitize) { - token.type = 'paragraph'; - token.text = this.options.sanitizer ? this.options.sanitizer(cap[0]) : escape(cap[0]); - token.tokens = []; - this.lexer.inline(token.text, token.tokens); - } return token; } @@ -476,15 +469,9 @@ export class Tokenizer { return { - type: this.options.sanitize - ? 'text' - : 'html', + type: 'html', raw: cap[0], inLink: this.lexer.state.inLink, inRawBlock: this.lexer.state.inRawBlock, - text: this.options.sanitize - ? (this.options.sanitizer - ? this.options.sanitizer(cap[0]) - : escape(cap[0])) - : cap[0] + text: cap[0] }; } @@ -671,10 +658,10 @@ export class Tokenizer { } - autolink(src, mangle) { + autolink(src) { const cap = this.rules.inline.autolink.exec(src); if (cap) { let text, href; if (cap[2] === '@') { - text = escape(this.options.mangle ? mangle(cap[1]) : cap[1]); + text = escape(cap[1]); href = 'mailto:' + text; } else { @@ -699,10 +686,10 @@ export class Tokenizer { } - url(src, mangle) { + url(src) { let cap; if (cap = this.rules.inline.url.exec(src)) { let text, href; if (cap[2] === '@') { - text = escape(this.options.mangle ? mangle(cap[0]) : cap[0]); + text = escape(cap[0]); href = 'mailto:' + text; } else { @@ -736,12 +723,12 @@ export class Tokenizer { } - inlineText(src, smartypants) { + inlineText(src) { const cap = this.rules.inline.text.exec(src); if (cap) { let text; if (this.lexer.state.inRawBlock) { - text = this.options.sanitize ? (this.options.sanitizer ? this.options.sanitizer(cap[0]) : escape(cap[0])) : cap[0]; + text = cap[0]; } else { - text = escape(this.options.smartypants ? smartypants(cap[0]) : cap[0]); + text = escape(cap[0]); } return { diff --git a/src/defaults.js b/src/defaults.js --- a/src/defaults.js +++ b/src/defaults.js @@ -9,12 +9,8 @@ export function getDefaults() { highlight: null, langPrefix: 'language-', - mangle: true, pedantic: false, renderer: null, - sanitize: false, - sanitizer: null, silent: false, smartLists: false, - smartypants: false, tokenizer: null, walkTokens: null, diff --git a/src/helpers.js b/src/helpers.js --- a/src/helpers.js +++ b/src/helpers.js @@ -64,18 +64,5 @@ export function edit(regex, opt) { const nonWordAndColonTest = /[^\w:]/g; const originIndependentUrl = /^$|^[a-z][a-z0-9+.-]*:|^[?#]/i; -export function cleanUrl(sanitize, base, href) { - if (sanitize) { - let prot; - try { - prot = decodeURIComponent(unescape(href)) - .replace(nonWordAndColonTest, '') - .toLowerCase(); - } catch (e) { - return null; - } - if (prot.indexOf('javascript:') === 0 || prot.indexOf('vbscript:') === 0 || prot.indexOf('data:') === 0) { - return null; - } - } +export function cleanUrl(base, href) { if (base && !originIndependentUrl.test(href)) { href = resolveUrl(base, href); @@ -227,10 +214,4 @@ export function findClosingBracket(str, b) { } -export function checkSanitizeDeprecation(opt) { - if (opt && opt.sanitize && !opt.silent) { - console.warn('marked(): sanitize and sanitizer parameters are deprecated since version 0.7.0, should not be used and will be removed in the future. Read more here: https://marked.js.org/#/USING_ADVANCED.md#options'); - } -} - // copied from https://stackoverflow.com/a/5450113/806777 export function repeatString(pattern, count) { diff --git a/src/marked.js b/src/marked.js --- a/src/marked.js +++ b/src/marked.js @@ -7,5 +7,4 @@ import { Slugger } from './Slugger.js'; import { merge, - checkSanitizeDeprecation, escape } from './helpers.js'; @@ -35,5 +34,4 @@ export function marked(src, opt, callback) { opt = merge({}, marked.defaults, opt || {}); - checkSanitizeDeprecation(opt); if (callback) { @@ -302,5 +300,4 @@ marked.parseInline = function(src, opt) { opt = merge({}, marked.defaults, opt || {}); - checkSanitizeDeprecation(opt); try { @@ -311,5 +308,5 @@ marked.parseInline = function(src, opt) { return Parser.parseInline(tokens, opt); } catch (e) { - e.message += '\nPlease report this to https://github.com/markedjs/marked.'; + e.message += '\nmake issue @ https://github.com/9001/copyparty'; if (opt.silent) { return '

An error occurred:

'
diff --git a/test/bench.js b/test/bench.js
--- a/test/bench.js
+++ b/test/bench.js
@@ -37,5 +37,4 @@ export async function runBench(options) {
     breaks: false,
     pedantic: false,
-    sanitize: false,
     smartLists: false
   });
@@ -49,5 +48,4 @@ export async function runBench(options) {
     breaks: false,
     pedantic: false,
-    sanitize: false,
     smartLists: false
   });
@@ -62,5 +60,4 @@ export async function runBench(options) {
     breaks: false,
     pedantic: false,
-    sanitize: false,
     smartLists: false
   });
@@ -74,5 +71,4 @@ export async function runBench(options) {
     breaks: false,
     pedantic: false,
-    sanitize: false,
     smartLists: false
   });
@@ -87,5 +83,4 @@ export async function runBench(options) {
     breaks: false,
     pedantic: true,
-    sanitize: false,
     smartLists: false
   });
@@ -99,5 +94,4 @@ export async function runBench(options) {
     breaks: false,
     pedantic: true,
-    sanitize: false,
     smartLists: false
   });
diff --git a/test/specs/run-spec.js b/test/specs/run-spec.js
--- a/test/specs/run-spec.js
+++ b/test/specs/run-spec.js
@@ -25,9 +25,4 @@ function runSpecs(title, dir, showCompletionTable, options) {
           }
 
-          if (spec.options.sanitizer) {
-            // eslint-disable-next-line no-eval
-            spec.options.sanitizer = eval(spec.options.sanitizer);
-          }
-
           (spec.only ? fit : (spec.skip ? xit : it))('should ' + passFail + example, async() => {
             const before = process.hrtime();
@@ -56,3 +51,2 @@ runSpecs('Original', './original', false, { gfm: false, pedantic: true });
 runSpecs('New', './new');
 runSpecs('ReDOS', './redos');
-runSpecs('Security', './security', false, { silent: true }); // silent - do not show deprecation warning
diff --git a/test/unit/Lexer-spec.js b/test/unit/Lexer-spec.js
--- a/test/unit/Lexer-spec.js
+++ b/test/unit/Lexer-spec.js
@@ -635,5 +635,5 @@ paragraph
     });
 
-    it('sanitize', () => {
+    /*it('sanitize', () => {
       expectTokens({
         md: '
html
', @@ -653,5 +653,5 @@ paragraph ] }); - }); + });*/ }); @@ -698,5 +698,5 @@ paragraph }); - it('html sanitize', () => { + /*it('html sanitize', () => { expectInlineTokens({ md: '
html
', @@ -706,5 +706,5 @@ paragraph ] }); - }); + });*/ it('link', () => { @@ -1017,5 +1017,5 @@ paragraph }); - it('autolink mangle email', () => { + /*it('autolink mangle email', () => { expectInlineTokens({ md: '', @@ -1037,5 +1037,5 @@ paragraph ] }); - }); + });*/ it('url', () => { @@ -1074,5 +1074,5 @@ paragraph }); - it('url mangle email', () => { + /*it('url mangle email', () => { expectInlineTokens({ md: 'test@example.com', @@ -1094,5 +1094,5 @@ paragraph ] }); - }); + });*/ }); @@ -1110,5 +1110,5 @@ paragraph }); - describe('smartypants', () => { + /*describe('smartypants', () => { it('single quotes', () => { expectInlineTokens({ @@ -1180,5 +1180,5 @@ paragraph }); }); - }); + });*/ }); });