From 785edf966812168c621de9841fa93d76e214cb3d Mon Sep 17 00:00:00 2001 From: Veronika Kabatova Date: Tue, 10 Apr 2018 16:30:44 +0200 Subject: Fix stuff around mbox header changes Bundle tests got broken after the subject in mbox was changed from the parsed version to the original one because the tests checked for the presence of patch's name in the response. Fixing this turned out to be a bit tricky since the tests check the mbox attachment and HTML responses separately, so we need a string that would be present in both (the intuitive idea of checking X-Patchwork-Id won't work well). Add the patch's name to the content of the test patch so we can continue testing things the same way, checking for the presence of patch's name. Also add a releasenote notifying about the inclusion of the original headers. Reverts: b2a25342 ("Use parsed subject for mboxes") Fixes: 01b9cbb9 ("Include all email headers in mboxes") Signed-off-by: Veronika Kabatova Reviewed-by: Stephen Finucane --- releasenotes/notes/mbox-headers-27e1f1bde9570e9f.yaml | 6 ++++++ 1 file changed, 6 insertions(+) create mode 100644 releasenotes/notes/mbox-headers-27e1f1bde9570e9f.yaml (limited to 'releasenotes/notes') diff --git a/releasenotes/notes/mbox-headers-27e1f1bde9570e9f.yaml b/releasenotes/notes/mbox-headers-27e1f1bde9570e9f.yaml new file mode 100644 index 0000000..7564118 --- /dev/null +++ b/releasenotes/notes/mbox-headers-27e1f1bde9570e9f.yaml @@ -0,0 +1,6 @@ +--- +other: + - | + mbox files now contain all headers from the original email. This also means + the ``Subject: `` header included will contain the original subject and not + the parsed Patchwork's version. -- cgit v1.2.3