Thread (5 messages) 5 messages, 3 authors, 2025-04-26

Re: [PATCH v6 1/1] send-email: retrieve Message-ID from outlook SMTP server

From: Aditya Garg <hidden>
Date: 2025-04-26 17:40:33

Possibly related (same subject, not in this thread)

Hi Eric> On Sat, Apr 26, 2025 at 4:37 AM Aditya Garg [off-list ref] wrote:
quoted
Add an option to allow users to specifically enable or disable
retrieving the Message-ID from the Outlook SMTP server. This can be
for other hosts mimicking the behaviour of Outlook, or for users who set
a custom domain to be a CNAME for the Outlook SMTP server.

Co-authored-by: Aditya Garg [off-list ref]
Signed-off-by: Aditya Garg <redacted>
---
diff --git a/Documentation/git-send-email.adoc b/Documentation/git-send-email.adoc
@@ -421,6 +421,21 @@ recipient's MUA.
+--[no-]smtp-outlook-id-tweak::
+       Outlook servers discard the Message-ID sent via email and assign a
+       new random Message-ID, thus breaking threads.
++
+--
+- '--smtp-outlook-id-tweak' will attempt to retrieve the ID from the server
+  irrespective of the SMTP server being used. Use only if Microsoft is your
+  email provider.
+- '--no-smtp-outlook-id-tweak' will disable this tweak irrespective of the
+  SMTP server being used.
+--
++
+If not sepcified, the default behaviour will be to enable the tweak only if the
+SMTP server is 'smtp.office365.com' or 'smtp-mail.outlook.com'.
s/sepcified/specified/
quoted
diff --git a/git-send-email.perl b/git-send-email.perl
@@ -60,6 +60,8 @@ sub usage {
     --smtp-encryption       <str>  * tls or ssl; anything else disables.
+    --smtp-outlook-id-tweak <0|1>  * This server munges Message-ID. Retrive it from
+                                     the server and assign to \$message_id.
s/Retrive/Retrieve/

As this is a user-facing help message, it seems unusual and unhelpful
for it to be talking about a variable ($message_id) which is internal
to the script. I realize that there is a slight precedent in which the
help for the --foo-cmd options talk about $patch_path, but those cases
are semantically different. Thus, it probably would be better to drop
mention of $message_id and just present a short and sweet explanation,
such as:

    Retrieve Message-ID from the server.
Thanks for the review, I would wait for a go ahead so that I can submit it
officially as well, with your suggestions incorporated :)
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help