diff options
author | Andrew Dolgov <[email protected]> | 2020-01-18 10:31:00 +0300 |
---|---|---|
committer | Andrew Dolgov <[email protected]> | 2020-01-18 10:31:00 +0300 |
commit | 303f8fb329ff75e41559806c9affd8c3c49504b0 (patch) | |
tree | 75391f026ce0fc060cd3ee3dadacb87fa6aa7e61 /js/Article.js | |
parent | 01513aa41b29d9b7c748ce86b8232c73901573bf (diff) |
properly escape quotes when rendering article data to html attributes via template strings
Diffstat (limited to 'js/Article.js')
-rw-r--r-- | js/Article.js | 8 |
1 files changed, 5 insertions, 3 deletions
diff --git a/js/Article.js b/js/Article.js index 08b565695..35b282d5d 100644 --- a/js/Article.js +++ b/js/Article.js @@ -156,14 +156,14 @@ define(["dojo/_base/declare"], function (declare) { comments_msg = hl.num_comments + " " + ngettext("comment", "comments", hl.num_comments) } - comments = `<a href="${hl.comments}">(${comments_msg})</a>`; + comments = `<a href="${escapeHtml(hl.comments)}">(${comments_msg})</a>`; } return comments; }, formatOriginallyFrom: function(hl) { return hl.orig_feed ? `<span> - ${__('Originally from:')} <a target="_blank" rel="noopener noreferrer" href="${hl.orig_feed[1]}">${hl.orig_feed[0]}</a> + ${__('Originally from:')} <a target="_blank" rel="noopener noreferrer" href="${escapeHtml(hl.orig_feed[1])}">${hl.orig_feed[0]}</a> </span>` : ""; }, unpack: function(row) { @@ -197,7 +197,9 @@ define(["dojo/_base/declare"], function (declare) { const article = `<div class="post post-${hl.id}"> <div class="header"> <div class="row"> - <div class="title"><a target="_blank" rel="noopener noreferrer" title="${hl.title}" href="${hl.link}">${hl.title}</a></div> + <div class="title"><a target="_blank" rel="noopener noreferrer" + title="${escapeHtml(hl.title)}" + href="${escapeHtml(hl.link)}">${hl.title}</a></div> <div class="date">${hl.updated_long}</div> </div> <div class="row"> |