Project

General

Profile

Actions
Copy link

Patch #3732

open

Incoming mail IMAP : Notify when an error occurs

Added by Thomas Recloux over 14 years ago. Updated over 11 years ago.

Status:
New
Priority:
Normal
Assignee:
-
Category:
Email receiving
Target version:
-
Start date:
2009-08-12
Due date:
% Done:

0%

Estimated time:

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

Files

Download all files
patch_incoming_mail_notify_error.diff (3.38 KB) patch_incoming_mail_notify_error.diff Thomas Recloux, 2009-08-12 16:18
patch_incoming_mail_notify_error_with_test.diff (4.42 KB) patch_incoming_mail_notify_error_with_test.diff Thomas Recloux, 2009-08-13 10:24
patch_incoming_mail_notify_sender.diff (793 Bytes) patch_incoming_mail_notify_sender.diff Mathieu Rochette, 2011-02-10 17:17
Actions
Copy link
 #1

Updated by Eric Davis over 14 years ago

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

Could you supply a test for the new behavior?

Actions
Copy link
 #2

Updated by Thomas Recloux over 14 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

Actions
Copy link
 #3

Updated by Lucas Panjer over 14 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

Actions
Copy link
 #4

Updated by Jean-Philippe Lang over 14 years ago

  • Category changed from Email notifications to Email receiving
Actions
Copy link
 #5

Updated by Stanislav German-Evtushenko almost 14 years ago

Does it work in 0.9-stable?

Actions
Copy link
 #6

Updated by Thomas Recloux almost 14 years ago

Yes, in production in my company.

Actions
Copy link
 #7

Updated by Stanislav German-Evtushenko almost 14 years ago

Thomas Recloux wrote:

Yes, in production in my company.

Thanks a lot!

Actions
Copy link
 #8

Updated by Patrick Cummins almost 14 years ago

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

Actions
Copy link
 #9

Updated by Eric Davis over 13 years ago

  • Assignee deleted (Eric Davis)
Actions
Copy link
 #10

Updated by Mathieu Rochette about 13 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.)

Actions
Copy link
 #11

Updated by Jean-Philippe Lang over 11 years ago

  • Status changed from 7 to New

Assigned issue with no assignee back to New status.

Actions
Copy link

Also available in: Atom PDF