From d26d36cc86c946bc9c784c0d0617426017bdd9bc Mon Sep 17 00:00:00 2001 From: Doug Coleman Date: Thu, 12 Sep 2019 18:13:13 -0500 Subject: [PATCH] smtp: Fix issues with sending an email using Outlook's smtp server. - Outlook requires saying HELO after upgrading to TLS. - Outlook stopped supporting plain auth in 2017. Tested on gmail and outlook. --- basis/smtp/smtp-docs.factor | 19 +++++++++++++++---- basis/smtp/smtp.factor | 20 +++++++++++++++++--- 2 files changed, 32 insertions(+), 7 deletions(-) diff --git a/basis/smtp/smtp-docs.factor b/basis/smtp/smtp-docs.factor index 4b72927d0f..c3cf0e5990 100644 --- a/basis/smtp/smtp-docs.factor +++ b/basis/smtp/smtp-docs.factor @@ -11,7 +11,7 @@ HELP: smtp-config { { $slot "server" } { "An " { $link } " of the SMTP server." } } { { $slot "tls?" } { "Secure socket after connecting to server, server must support " { $snippet "STARTTLS" } } } { { $slot "read-timeout" } { "Length of time after which we give up waiting for a response." } } - { { $slot "auth" } { "Either " { $link no-auth } " or an instance of " { $link plain-auth } } } + { { $slot "auth" } { "Either " { $link no-auth } " or an instance of " { $link plain-auth } " or " { $link login-auth } } } } } ; @@ -25,12 +25,22 @@ HELP: no-auth { $class-description "If the " { $snippet "auth" } " slot is set to this value, no authentication will be performed." } ; HELP: plain-auth -{ $class-description "If the " { $snippet "auth" } " slot is set to this value, plain authentication will be performed, with the username and password stored in the " { $slot "username" } " and " { $slot "password" } " slots of the tuple sent to the server as plain-text." } ; +{ $class-description "If the " { $snippet "auth" } " slot is set to this value, plain authentication will be performed, with the username and password stored in the " { $slot "username" } " and " { $slot "password" } " slots of the tuple sent to the server as plain-text." } +{ $notes "This authentication method is no longer supported by Outlook mail servers." } ; HELP: { $values { "username" string } { "password" string } { "plain-auth" plain-auth } } { $description "Creates a new " { $link plain-auth } " instance." } ; +HELP: login-auth +{ $class-description "If the " { $snippet "auth" } " slot is set to this value, LOGIN authentication will be performed, with the username and password stored in the " { $slot "username" } " and " { $slot "password" } " slots of the tuple sent to the server as plain-text." } ; + +HELP: +{ $values { "username" string } { "password" string } { "login-auth" login-auth } } +{ $description "Creates a new " { $link login-auth } " instance." } ; + +{ no-auth plain-auth login-auth } related-words + HELP: with-smtp-config { $values { "quot" quotation } } { $description "Connects to an SMTP server using credentials and settings stored in " { $link smtp-config } " and calls the " { $link with-smtp-connection } " combinator." } @@ -84,7 +94,7 @@ HELP: send-email ARTICLE: "smtp-gmail" "Setting up SMTP with gmail" "If you plan to send all email from the same address, then setting the config variable in the global namespace is the best option. The code example below does this approach and is meant to go in your " { $link ".factor-boot-rc" } "." $nl -"First, we set the login and password to a " { $link } " tuple with our login. Next, we set the gmail server address with an " { $link } " object. Finally, we tell the SMTP library to use a secure connection." +"First, we set the login and password to a " { $link } " tuple with our login. Next, we set the gmail server address with an " { $link } " object. Finally, we tell the SMTP library to use a secure connection." { $notes "Observed on 2016-03-02: Gmail has restrictions for what they consider \"less secure apps\" (these include the factor smtp client)." { $list @@ -99,7 +109,7 @@ ARTICLE: "smtp-gmail" "Setting up SMTP with gmail" "default-smtp-config \"smtp.gmail.com\" 587 >>server t >>tls? - \"my.gmail.address@gmail.com\" \"qwertyuiasdfghjk\" >>auth + \"my.gmail.address@gmail.com\" \"qwertyuiasdfghjk\" >>auth \\ smtp-config set-global" } } ; @@ -117,6 +127,7 @@ $nl { $subsections no-auth plain-auth + login-auth } "Constructing an e-mail:" { $subsections diff --git a/basis/smtp/smtp.factor b/basis/smtp/smtp.factor index 043d6d81dc..e00c3e2b41 100644 --- a/basis/smtp/smtp.factor +++ b/basis/smtp/smtp.factor @@ -17,6 +17,9 @@ SINGLETON: no-auth TUPLE: plain-auth username password ; C: plain-auth +TUPLE: login-auth username password ; +C: login-auth + : ( -- smtp-config ) smtp-config new ; inline @@ -129,7 +132,7 @@ ERROR: smtp-transaction-failed < smtp-error ; : check-response ( response -- ) dup code>> { - { [ dup { 220 235 250 221 354 } member? ] [ 2drop ] } + { [ dup { 220 235 250 221 334 354 } member? ] [ 2drop ] } { [ dup 400 499 between? ] [ drop smtp-server-busy ] } { [ dup 500 = ] [ drop smtp-syntax-error ] } { [ dup 501 = ] [ drop smtp-command-not-implemented ] } @@ -149,13 +152,21 @@ GENERIC: send-auth ( auth -- ) M: no-auth send-auth drop ; +: >smtp-base64 ( str -- str' ) + utf8 encode >base64 >string ; + : plain-auth-string ( username password -- string ) - [ "\0" prepend ] bi@ append utf8 encode >base64 >string ; + [ "\0" prepend ] bi@ append >smtp-base64 ; M: plain-auth send-auth [ username>> ] [ password>> ] bi plain-auth-string "AUTH PLAIN " prepend command get-ok ; +M: login-auth send-auth + "AUTH LOGIN" command get-ok + [ username>> >smtp-base64 command get-ok ] + [ password>> >smtp-base64 command get-ok ] bi ; + : auth ( -- ) smtp-config get auth>> send-auth ; : encode-header ( string -- string' ) @@ -214,7 +225,10 @@ ERROR: invalid-header-string string ; [ get-ok helo get-ok - smtp-config get tls?>> [ start-tls get-ok send-secure-handshake ] when + smtp-config get tls?>> [ + start-tls get-ok send-secure-handshake + helo get-ok + ] when auth dup from>> extract-email mail-from get-ok dup to>> [ extract-email rcpt-to get-ok ] each