A common setup is to use crontab to schedule drush cron via drush.

In this case, failure to send mail during cron causes error messages, which causes drush to return an error code, which can cause cron to send an email to the sysadmin.

This level of alerting is unwarranted, especially as for a custom mail transport such as SwiftMailer sending via SMTP, failure to send mail can occur just for sending to an unknown mailbox.

See core issue #2349725: Core Mail class prints non-overridable errors on the page

Also swiftmailer issue #3057200: Duplicate error message

CommentFileSizeAuthor
#14 simplenews.no-display-error.ONLYTESTS.3053223-11.patch2.23 KBadamps
#12 simplenews.no-display-error.3053223-11.patch2.71 KBadamps
#11 simplenews.no-display-error.3053223-interdiff-7-11.txt410 bytesadamps
#11 simplenews.no-display-error.3053223-11.patch2.71 KBadamps
#7 simplenews.no-display-error.3053223-interdiff-5-7.txt1.88 KBadamps
#7 simplenews.no-display-error.3053223-7.patch2.43 KBadamps
#7 simplenews.no-display-error.ONLYTESTS.3053223-7.patch1.95 KBadamps
#5 simplenews.no-display-error.3053223-5.patch485 bytesadamps

Comments

AdamPS created an issue. See original summary.

adamps’s picture

adamps commented
Issue summary: View changes
Related issues: +#2349725: Core Mail class prints non-overridable errors on the page
adamps’s picture

adamps commented
Priority: Normal » Major
adamps’s picture

adamps commented
Issue summary: View changes
Related issues: +#3057200: Duplicate error message
adamps’s picture

adamps commented
Status: Active » Needs review
Issue tags: +Needs tests
StatusFileSize
new simplenews.no-display-error.3053223-5.patch485 bytes

Status: Needs review » Needs work

The last submitted patch, 5: simplenews.no-display-error.3053223-5.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

adamps’s picture

adamps commented
Status: Needs work » Needs review
Issue tags: -Needs tests
StatusFileSize
new simplenews.no-display-error.ONLYTESTS.3053223-7.patch1.95 KB
new simplenews.no-display-error.3053223-7.patch2.43 KB
new simplenews.no-display-error.3053223-interdiff-5-7.txt1.88 KB
adamps’s picture

adamps commented
Version: 8.x-1.x-dev » 8.x-2.x-dev

The last submitted patch, 7: simplenews.no-display-error.ONLYTESTS.3053223-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

Status: Needs review » Needs work

The last submitted patch, 7: simplenews.no-display-error.3053223-7.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

adamps’s picture

adamps commented
Version: 8.x-2.x-dev » 8.x-1.x-dev
StatusFileSize
new simplenews.no-display-error.3053223-11.patch2.71 KB
new simplenews.no-display-error.3053223-interdiff-7-11.txt410 bytes
adamps’s picture

adamps commented
Version: 8.x-1.x-dev » 8.x-2.x-dev
Status: Needs work » Needs review
StatusFileSize
new simplenews.no-display-error.3053223-11.patch2.71 KB

Status: Needs review » Needs work

The last submitted patch, 12: simplenews.no-display-error.3053223-11.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

adamps’s picture

adamps commented
StatusFileSize
new simplenews.no-display-error.ONLYTESTS.3053223-11.patch2.23 KB
adamps’s picture

adamps commented
Status: Needs work » Needs review

The last submitted patch, 11: simplenews.no-display-error.3053223-11.patch, failed testing. View results
- codesniffer_fixes.patch Interdiff of automated coding standards fixes only.

adamps’s picture

adamps commented
Issue tags: +Plan to commit

Patch relies on D8.8 to work, but the code is harmless in D8.7.

  • AdamPS committed f7618b4 on 8.x-2.x 
    Issue #3053223 by AdamPS: Failure to send mail during cron causes errors...
adamps’s picture

adamps commented
Status: Needs review » Fixed
Issue tags: -Plan to commit

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.