[Tickets #12659] Re: Horde SMTP sends empty EMail if PIPELINING is used...

[noreply at bugs.horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/12659
------------------------------------------------------------------------------
  Ticket             | 12659
  Updated By         | Git Commit <commits at lists.horde.org>
  Summary            | Horde SMTP sends empty EMail if PIPELINING is used...
  Queue              | Horde Framework Packages
  Version            | Git master
  Type               | Bug
  State              | Resolved
  Priority           | 2. Medium
  Milestone          |
  Patch              |
  Owners             | Michael Slusarz
------------------------------------------------------------------------------


Git Commit <commits at lists.horde.org> (2013-12-17 11:35) wrote:

Changes have been made in Git (master):

commit 1c5aa652d33512965a3b6a02ba407e5214a7edbd
Author: Michael M Slusarz <slusarz at horde.org>
Date:   Thu Sep 5 10:41:43 2013 -0600

     [mms] Fix aborting a pipelined DATA command if only a subset of  
recipients were rejected (Bug #12659).

     See: http://www.imc.org/ietf-smtp/mail-archive/msg02687.html

     The RFC 2920 text is a bit vague.  Servers must issue a 554 if ALL
     recipients are rejected.  This might not happen until after the DATA
     command is sent though (which is why we were sending empty DATA data).
     Unfortunately, *partial* rejection of recipients will NOT abort the DATA
     command. Since we don't pre-verify mail recipients, we want to catch the
     case if ANY recipients are bad, since the user generally is expecting
     that all e-mail addresses provided will have the message sent to.  This
     means we can't pipeline DATA.

  framework/Smtp/lib/Horde/Smtp.php |   49  
++++++++++++++++++-------------------
  framework/Smtp/package.xml        |    2 +
  2 files changed, 26 insertions(+), 25 deletions(-)

http://git.horde.org/horde-git/-/commit/1c5aa652d33512965a3b6a02ba407e5214a7edbd