# [StGit PATCH] mail: Ask for the SMTP credentials before sending the messages 4 messages from 2010-02-12 to 2010-03-01. Participants: Catalin Marinas, Pavel Roskin. Thread: https://gitlist.dev/t/22631 ## Catalin Marinas, 2010-02-12 16:11 Subject: [StGit PATCH] mail: Ask for the SMTP credentials before sending the messages Message-ID: <20100212161150.12036.62592.stgit@pc1117.cambridge.arm.com> URL: https://gitlist.dev/e/20100212161150.12036.62592.stgit%40pc1117.cambridge.arm.com ``` The original implementation was asking for the SMTP password on every patch sent. This patch only asks the password once before sending or even editing the cover message and patches. Signed-off-by: Catalin Marinas Cc: Pavel Roskin Cc: Alex Chiang --- stgit/commands/mail.py | 22 ++++++++++++++++++++-- 1 files changed, 20 insertions(+), 2 deletions(-) diff --git a/stgit/commands/mail.py b/stgit/commands/mail.py index 34262d3..b0a8f13 100644 --- a/stgit/commands/mail.py +++ b/stgit/commands/mail.py @@ -192,9 +192,18 @@ def __send_message_sendmail(sendmail, msg): cmd = sendmail.split() Run(*cmd).raw_input(msg).discard_output() -def __send_message_smtp(smtpserver, from_addr, to_addr_list, msg, options): - """Send the message using the given SMTP server +__smtp_credentials = None + +def __set_smtp_credentials(options): + """Set the (smtpuser, smtppassword, smtpusetls) credentials if the method + of sending is SMTP. """ + global __smtp_credentials + + smtpserver = options.smtp_server or config.get('stgit.smtpserver') + if options.git or smtpserver.startswith('/'): + return + smtppassword = options.smtp_password or config.get('stgit.smtppassword') smtpuser = options.smtp_user or config.get('stgit.smtpuser') smtpusetls = options.smtp_tls or config.get('stgit.smtptls') == 'yes' @@ -206,6 +215,13 @@ def __send_message_smtp(smtpserver, from_addr, to_addr_list, msg, options): if (smtpuser and not smtppassword): smtppassword = getpass.getpass("Please enter SMTP password: ") + __smtp_credentials = (smtpuser, smtppassword, smtpusetls) + +def __send_message_smtp(smtpserver, from_addr, to_addr_list, msg, options): + """Send the message using the given SMTP server + """ + smtpuser, smtppassword, smtpusetls = __smtp_credentials + try: s = smtplib.SMTP(smtpserver) except Exception, err: @@ -652,6 +668,8 @@ def func(parser, options, args): else: ref_id = None + # get username/password if sending by SMTP + __set_smtp_credentials(options) # send the cover message (if any) if options.cover or options.edit_cover: ``` ## Pavel Roskin, 2010-02-12 22:42 Subject: Re: [StGit PATCH] mail: Ask for the SMTP credentials before sending the messages Message-ID: <1266014578.25535.9.camel@mj> URL: https://gitlist.dev/e/1266014578.25535.9.camel%40mj In-Reply-To: <20100212161150.12036.62592.stgit@pc1117.cambridge.arm.com> ``` On Fri, 2010-02-12 at 16:11 +0000, Catalin Marinas wrote: > The original implementation was asking for the SMTP password on every > patch sent. This patch only asks the password once before sending or > even editing the cover message and patches. > > Signed-off-by: Catalin Marinas > Cc: Pavel Roskin > Cc: Alex Chiang Tested-by: Pavel Roskin -- Regards, Pavel Roskin ``` ## Pavel Roskin, 2010-02-12 22:56 Subject: Re: [StGit PATCH] mail: Ask for the SMTP credentials before sending the messages Message-ID: <1266015411.25535.21.camel@mj> URL: https://gitlist.dev/e/1266015411.25535.21.camel%40mj In-Reply-To: <20100212161150.12036.62592.stgit@pc1117.cambridge.arm.com> ``` On Fri, 2010-02-12 at 16:11 +0000, Catalin Marinas wrote: > The original implementation was asking for the SMTP password on every > patch sent. This patch only asks the password once before sending or > even editing the cover message and patches. I don't have time to investigate at the moment, but "make test" fails in t1900-mail.sh on the "proposed" branch, which includes this patch: *** t1900-mail.sh *** * ok 1: Initialize the StGIT repository * ok 2: Put all the patches in an mbox * ok 3: Import the mbox and compare * FAIL 4: Put all the patches in an mbox with patch attachments stg mail --to="Inge Ström " -a -m \ -t ../../templates/mailattch.tmpl > mbox1 * FAIL 5: Import the mbox containing patch attachments and compare t1=$(git cat-file -p $(stg id) | grep ^tree) stg pop -a && stg import -M mbox1 && t2=$(git cat-file -p $(stg id) | grep ^tree) && [ "$t1" = "$t2" ] * FAIL 6: Check the To:, Cc: and Bcc: headers stg mail --to=a@a --cc="b@b, c@c" --bcc=d@d $(stg top) -m \ -t ../../templates/patchmail.tmpl > mbox && test "$(cat mbox | grep -e "^To:")" = "To: a@a" && test "$(cat mbox | grep -e "^Cc:")" = "Cc: b@b, c@c" && test "$(cat mbox | grep -e "^Bcc:")" = "Bcc: d@d" * FAIL 7: Check the --auto option stg edit --sign && stg mail --to=a@a --cc="b@b, c@c" --bcc=d@d --auto $(stg top) -m \ -t ../../templates/patchmail.tmpl > mbox && test "$(cat mbox | grep -e "^To:")" = "To: a@a" && test "$(cat mbox | grep -e "^Cc:")" = \ "Cc: C O Mitter , b@b, c@c" && test "$(cat mbox | grep -e "^Bcc:")" = "Bcc: d@d" * FAIL 8: Check the e-mail address duplicates stg mail --to="a@a, b b " --cc="b@b, c@c" \ --bcc="c@c, d@d, committer@example.com" --auto $(stg top) -m \ -t ../../templates/patchmail.tmpl > mbox && test "$(cat mbox | grep -e "^To:")" = "To: b b , a@a" && test "$(cat mbox | grep -e "^Cc:")" = "Cc: c@c" && test "$(cat mbox | grep -e "^Bcc:")" = "Bcc: committer@example.com, d@d" * failed 5 among 8 test(s) make[1]: *** [t1900-mail.sh] Error 1 make[1]: Leaving directory `/home/proski/src/stgit/t' make: *** [test] Error 2 -- Regards, Pavel Roskin ``` ## Catalin Marinas, 2010-03-01 15:46 Subject: Re: [StGit PATCH] mail: Ask for the SMTP credentials before sending the messages Message-ID: URL: https://gitlist.dev/e/b0943d9e1003010746l1d4208bas48fcfba36efa5605%40mail.gmail.com In-Reply-To: <1266015411.25535.21.camel@mj> ``` On 12 February 2010 22:56, Pavel Roskin wrote: > On Fri, 2010-02-12 at 16:11 +0000, Catalin Marinas wrote: >> The original implementation was asking for the SMTP password on every >> patch sent. This patch only asks the password once before sending or >> even editing the cover message and patches. > > I don't have time to investigate at the moment, but "make test" fails in > t1900-mail.sh on the "proposed" branch, which includes this patch: The patch was converting the message headers to Header objects rather than strings so that the long-line folding is done using "\n " rather than "\n\t" (Outlook cannot handle the latter correctly). But it looks like the Python library assumes that they are strings and getting "content-type" fails (I would say Python bug). Anyway, I change the patch so that it only converts the the Subject header which is the one that bothers me most: commit 1b0c0113861681974b8905dbe10a57f6831ecb87 Author: Catalin Marinas Date: Fri Feb 12 15:36:37 2010 +0000 mail: Use space rather than tab for long subject header folding The default Python implementation (at least 2.5 and earlier) fold long e-mail header lines by inserting "\n\t". This causes issues with some e-mail clients that remove both "\n\t". The RFC2822 shows that folding should be done with "\n ". The Python workaround is to use a Header object instead of a string when setting the message headers. Signed-off-by: Catalin Marinas diff --git a/stgit/commands/mail.py b/stgit/commands/mail.py index d0334b4..ed55fd9 100644 --- a/stgit/commands/mail.py +++ b/stgit/commands/mail.py @@ -426,6 +426,13 @@ def __encode_message(msg): new_val = ' '.join(words) msg.replace_header(header, new_val) + # replace the Subject string with a Header() object otherwise the long + # line folding is done using "\n\t" rather than "\n ", causing issues with + # some e-mail clients + subject = msg.get('subject', '') + msg.replace_header('subject', + email.Header.Header(subject, header_name = 'subject')) + # encode the body and set the MIME and encoding headers if msg.is_multipart(): for p in msg.get_payload(): -- Catalin ```