From b9d9db88fffad60798397b006445e505f6b5e447 Mon Sep 17 00:00:00 2001
From: Magnus Hagander <magnus@hagander.net>
Date: Tue, 8 Dec 2020 16:32:04 +0100
Subject: [PATCH] Properly escape list of messages when creating a new entry

This has been broken since day 1... oops.

Noted by Vik Fearing
---
 pgcommitfest/commitfest/static/commitfest/js/commitfest.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pgcommitfest/commitfest/static/commitfest/js/commitfest.js b/pgcommitfest/commitfest/static/commitfest/js/commitfest.js
index 69c41ec..2f1edda 100644
--- a/pgcommitfest/commitfest/static/commitfest/js/commitfest.js
+++ b/pgcommitfest/commitfest/static/commitfest/js/commitfest.js
@@ -20,7 +20,7 @@ function findLatestThreads() {
          sel = $('#attachThreadList');
          sel.find('option').remove();
          $.each(data, function(m,i) {
-            sel.append('<option value="' + i.msgid + '">' + i.from + ': ' + i.subj + ' (' + i.date + ')</option>');
+             sel.append($('<option/>').text(i.from + ': ' + i.subj + ' (' + i.date + ')').val(i.msgid));
          });
    }).always(function() {
       $('#attachThreadListWrap').removeClass('loading');
-- 
2.39.5