Feature #38273
closed
Improve errors in MailHandler: add MissingContainer and LockedTopic exception
Added by Felix Schäfer almost 2 years ago.
Updated over 1 year ago.
Description
There are some cases for which the MailHandler
drops incoming emails and just logs an info. This patch replaces this with a custom error to allow for plugins to more easily hook into those unhandled cases. This also unifies the logged message and the error handling in MailHandler
.
Files
The attached patch adds a custom generic Error
for cases when the replied to object/the container object cannot be found, either because it was deleted or because the e-mail comes from another Redmine installation with different IDs. The second patch adds a custom Error
for cases where the topic the e-mail answers to is locked.
- Blocks Feature #38274: Receive e-mail replies to news and news comments added
- Target version set to 5.1.0
Setting the target version to 5.1.0.
- File 1-missing_container_error-v2.patch added
Updated the first patch to apply cleanly to the current trunk.
Go MAEDA wrote:
Updated the first patch to apply cleanly to the current trunk.
I am withdrawing this patch. 1-missing_container_error.patch can be applied without problems if #38263 is applied.
- File deleted (
1-missing_container_error-v2.patch)
- Blocked by Feature #38263: Try importing journal replies as issue reply where applicable added
- Subject changed from Improve errors in MailHandler to Improve errors in MailHandler: add MissingContainer and LockedTopic exception
- Status changed from New to Closed
- Assignee set to Go MAEDA
- Resolution set to Fixed
Committed the patches. Thank you.
Also available in: Atom
PDF