1713724 - Should abort sending on RCPT error in SmtpClient.jsm

Status: RESOLVED FIXED

(MailNews Core :: Networking: SMTP, defect)

Description

[Ping Chen [:rnons] (away)]

+++ This bug was initially created as a clone of Bug #1710066 +++

The original emailjs-smtp-client treats RCPT error as soft error, as long as one RCPT succeeded, it will send the mail. We should notify user whenever a RCPT error is received.

Comment 1

[Ping Chen [:rnons] (away)]

Attachment: Bug 1713724 - Abort sending on RCPT error in SmtpClient.jsm. r=mkmelin

Comment 2

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/644798fad6f7
Abort sending on RCPT error in SmtpClient.jsm. r=mkmelin

Comment 3

[Ping Chen [:rnons] (away)]

Comment on attachment 9224378 [details]
Bug 1713724 - Abort sending on RCPT error in SmtpClient.jsm. r=mkmelin

[Approval Request Comment]
Regression caused by (bug #): bug 1661694
User impact if declined: When sending to multiple recipients, sending is not aborted on RCPT error, user won't notice some recipients failed.
Testing completed (on c-c, etc.): c-c
Risk to taking this patch (and alternatives if risky): low

Comment 4

[Wayne Mery (:wsmwk)]

Comment on attachment 9224378 [details]
Bug 1713724 - Abort sending on RCPT error in SmtpClient.jsm. r=mkmelin

[Triage Comment]
Approved for beta

Comment 5

[Rob Lemley [:rjl]]

Thunderbird 90.0b2:
https://hg.mozilla.org/releases/comm-beta/rev/893903de49ce