Patch #3732

Incoming mail IMAP : Notify when an error occurs

Added by Thomas Recloux about 12 years ago. Updated over 8 years ago.

Status:NewStart date:2009-08-12
Priority:NormalDue date:
Assignee:-% Done:

0%

Category:Email receiving
Target version:-

Description

Add the possibility to notify the administrators when an error occurs during incoming mail parsing.

Add a notify_on_failure option to the tasks.

Built against the trunk in revision 2836.
Tested on trunk and 0.8.4

patch_incoming_mail_notify_error.diff Magnifier (3.38 KB) Thomas Recloux, 2009-08-12 16:18

patch_incoming_mail_notify_error_with_test.diff Magnifier (4.42 KB) Thomas Recloux, 2009-08-13 10:24

patch_incoming_mail_notify_sender.diff Magnifier (793 Bytes) Mathieu Rochette, 2011-02-10 17:17

History

#1 Updated by Eric Davis about 12 years ago

  • Status changed from New to 7
  • Assignee set to Eric Davis

Could you supply a test for the new behavior?

#2 Updated by Thomas Recloux about 12 years ago

I added a test for the mailer.

I do not know how to test the rake task and the Imap class.

This patch replaces the first one

Thanks

#3 Updated by Lucas Panjer about 12 years ago

It would be good if there was a general solution that also covered failures for emails piped in using extra/mail_handler/rdm_mailhandler.rb

#4 Updated by Jean-Philippe Lang almost 12 years ago

  • Category changed from Email notifications to Email receiving

#5 Updated by Stanislav German-Evtushenko over 11 years ago

Does it work in 0.9-stable?

#6 Updated by Thomas Recloux over 11 years ago

Yes, in production in my company.

#7 Updated by Stanislav German-Evtushenko over 11 years ago

Thomas Recloux wrote:

Yes, in production in my company.

Thanks a lot!

#8 Updated by Patrick Cummins about 11 years ago

How would I modify the patch to also notify the person sending the emails?

#9 Updated by Eric Davis almost 11 years ago

  • Assignee deleted (Eric Davis)

#10 Updated by Mathieu Rochette over 10 years ago

Patrick Cummins wrote:

How would I modify the patch to also notify the person sending the emails?

I was also intersted on that, just apply the attachd patch on top of Thomas Recloux's

once applied, mail are sent only to the sender and not to admins.
maybe someon with ruby skills could works something wich would allow something like notify_on_failure=admins,sender

I think it would be great to integrate that int redmine because it's really a show-stopper if the sender can't know that his mail has been ignored.

for now, the mail just tell, the received mail could not been processed. it might be intersting to explain why. eg: the message logged (unknown mail address, not authorized, missing required field, etc.)

#11 Updated by Jean-Philippe Lang over 8 years ago

  • Status changed from 7 to New

Assigned issue with no assignee back to New status.

Also available in: Atom PDF