parser: subject_check needs to call clean_header

When testing with headers with unencoded characters on Py3,
subject_check would fail. It was getting an Header class, rather
than a str/bytes, and so the regex was failing.

Run clean_header to decode everything fully first.

Signed-off-by: Daniel Axtens <dja@axtens.net>
Reviewed-by: Stephen Finucane <stephen@that.guru>

diff --git a/patchwork/parser.py b/patchwork/parser.py
index 3619a07ef725da4ea8ffa481b1164fa17a8b040c..db3b898c464839e6a58bd14f2b69e7971958e826 100644 (file)
--- a/patchwork/parser.py
+++ b/patchwork/parser.py
@@ -423,7 +423,7 @@ def subject_check(subject):
     """Determine if a mail is a reply."""
     comment_re = re.compile(r'^(re)[:\s]\s*', re.I)
 
-    return comment_re.match(subject)
+    return comment_re.match(clean_header(subject))
 
 
 def clean_content(content):