We need to get emails when an error occurs on a site.

Emails are only being sent when an exception occurs, but it would be helpful if emails would be sent when even an error occurs on the site.

CommentFileSizeAuthor
#12 interdiff_11_12.patch4.47 KBkosa ilma
#12 3050402-extending-module-to-sending-emails-on-errors-too-12.patch14.39 KBkosa ilma
#11 3050402-extending-module-to-sending-emails-on-errors-too-11.patch15.84 KBkosa ilma
#8 3050402-extending-module-to-sending-emails-on-errors-too-8.patch15.61 KBkosa ilma
#7 3050402-extending-module-to-sending-emails-on-errors-too-7.patch15.56 KBkosa ilma
#5 3050402-extending-module-to-sending-emails-on-errors-too-5.patch15.26 KBkosa ilma

Comments

Kosa Ilma created an issue. See original summary.

kosa ilma’s picture

kosa ilma commented
Issue summary: View changes
kosa ilma’s picture

kosa ilma commented
Category: Task » Feature request
pedrop’s picture

pedrop commented

by default only send emails on severity Error and higher.
Later would nice to make this configurable.

kosa ilma’s picture

kosa ilma commented
StatusFileSize
new 3050402-extending-module-to-sending-emails-on-errors-too-5.patch15.26 KB

The patch contains extending the mail with extra info and some PHPCS fix too.
The severity for errors is configurable, by default: emergency, alert, critical and error are set.

kosa ilma’s picture

kosa ilma commented
Assigned: kosa ilma » Unassigned
Status: Active » Needs review
kosa ilma’s picture

kosa ilma commented
StatusFileSize
new 3050402-extending-module-to-sending-emails-on-errors-too-7.patch15.56 KB

Minor fixes in the configuration form for errors' severity and some PHPCS fix too.

kosa ilma’s picture

kosa ilma commented
StatusFileSize
new 3050402-extending-module-to-sending-emails-on-errors-too-8.patch15.61 KB

Minor fixes in the mail.

szato’s picture

szato commented
Status: Needs review » Reviewed & tested by the community

Tested patch #8 with core 8.6.16 and 8.7.4. Works for me.

pedrop’s picture

pedrop commented
Assigned: Unassigned » kosa ilma
Status: Reviewed & tested by the community » Needs work

use constants for error severity instead of strings

kosa ilma’s picture

kosa ilma commented
StatusFileSize
new 3050402-extending-module-to-sending-emails-on-errors-too-11.patch15.84 KB

The new patch file using the constants (fixed the newly created files too).

kosa ilma’s picture

kosa ilma commented
StatusFileSize
new 3050402-extending-module-to-sending-emails-on-errors-too-12.patch14.39 KB
new interdiff_11_12.patch4.47 KB

Refactored patch.

  • Kosa Ilma committed 168efe5 on 8.x-3.x 
    Issue #3050402 by Kosa Ilma, pedrop, szato: Extending module to sending...
kosa ilma’s picture

kosa ilma commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

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