From e27ff061dc01e51967a978884a5c59152863ab9c Mon Sep 17 00:00:00 2001 From: Yuri Volchkov Date: Wed, 20 Jun 2018 14:21:42 +0200 Subject: parsemail: ignore html part of multi-part comments Currently an html-protection present only for patch-emails. If a multi-part comment-email arrives, it messes up patchwork. In my case, the symptom was a non intended 'Signed-off-by' in the downloaded patches, with html-like junk. This patch makes parsemail skip all parts of comment which are not text/plain. Of course, this will drop html-only emails completely. But they can not be parsed anyways. Signed-off-by: Yuri Volchkov Reviewed-by: Stephen Finucane --- patchwork/parser.py | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/patchwork/parser.py b/patchwork/parser.py index 8f9af81..a40f931 100644 --- a/patchwork/parser.py +++ b/patchwork/parser.py @@ -576,10 +576,13 @@ def find_comment_content(mail): """Extract content from a mail.""" commentbuf = '' - for payload, _ in _find_content(mail): + for payload, subtype in _find_content(mail): if not payload: continue + if subtype != 'plain': + continue + commentbuf += payload.strip() + '\n' commentbuf = clean_content(commentbuf) -- cgit v1.2.3