diff --git a/webapp/tests/formatting_links.test.jsx b/webapp/tests/formatting_links.test.jsx index 237ef61215..30461e6be8 100644 --- a/webapp/tests/formatting_links.test.jsx +++ b/webapp/tests/formatting_links.test.jsx @@ -501,4 +501,18 @@ describe('Markdown.Links', function() { done(); }); + + it('Links containing %', function(done) { + assert.equal( + Markdown.format('https://en.wikipedia.org/wiki/%C3%89').trim(), + '

https://en.wikipedia.org/wiki/%C3%89

' + ); + + assert.equal( + Markdown.format('https://en.wikipedia.org/wiki/%E9').trim(), + '

https://en.wikipedia.org/wiki/%E9

' + ); + + done(); + }); }); diff --git a/webapp/utils/markdown.jsx b/webapp/utils/markdown.jsx index 7f597eb3d4..0b279ca6dc 100644 --- a/webapp/utils/markdown.jsx +++ b/webapp/utils/markdown.jsx @@ -135,7 +135,13 @@ class MattermostMarkdownRenderer extends marked.Renderer { let outHref = href; try { - const unescaped = decodeURIComponent(unescape(href)).replace(/[^\w:]/g, '').toLowerCase(); + let unescaped = unescape(href); + try { + unescaped = decodeURIComponent(unescaped); + } catch (e) { + unescaped = global.unescape(unescaped); + } + unescaped = unescaped.replace(/[^\w:]/g, '').toLowerCase(); if (unescaped.indexOf('javascript:') === 0 || unescaped.indexOf('vbscript:') === 0 || unescaped.indexOf('data:') === 0) { // eslint-disable-line no-script-url return text;