From 465c25ce2ddb310b6faf7fc27ab6caa2934337ee Mon Sep 17 00:00:00 2001 From: Trevor Gamblin Date: Fri, 9 Feb 2024 09:30:41 -0500 Subject: patchtest-send-results: Add 'References' header Based on https://www.rfc-editor.org/rfc/rfc2822#section-3.6.4, to get proper threading there should be a 'References' field defined in Patchtest's response headers if its replies are going to thread properly on Patchwork. This wasn't there before, hence why were were seeing the responses in the right place on the mailing list but not Patchwork. Since we only want Patchtest to reply directly to the tested patch, the 'References' field should only need to contain the same ID as the 'In-Reply-To' one. (From OE-Core rev: f8a2af973d829c3c7f2dce39ce452daf74f31f52) Signed-off-by: Trevor Gamblin Signed-off-by: Richard Purdie --- scripts/patchtest-send-results | 5 +++++ 1 file changed, 5 insertions(+) (limited to 'scripts/patchtest-send-results') diff --git a/scripts/patchtest-send-results b/scripts/patchtest-send-results index 075b60d8f7..9fd24c9b95 100755 --- a/scripts/patchtest-send-results +++ b/scripts/patchtest-send-results @@ -81,9 +81,14 @@ if has_a_failed_test(testresult): reply_contents = greeting + testresult + suggestions ses_client = boto3.client('ses', region_name='us-west-2') + + # Construct the headers for the email. We only want to reply + # directly to the tested patch, so make In-Reply-To and References + # the same value. raw_data = 'From: ' + from_address + '\nTo: ' + reply_address + \ '\nCC: ' + cc_address + '\nSubject:' + subject_line + \ '\nIn-Reply-To:' + in_reply_to + \ + '\nReferences:' + in_reply_to + \ '\nMIME-Version: 1.0" + \ "\nContent-type: Multipart/Mixed;boundary="NextPart"\n\n--NextPart\nContent-Type: text/plain\n\n' + \ reply_contents + '\n\n--NextPart' -- cgit v1.2.3-54-g00ecf