Feature #38273
closed
Improve errors in MailHandler: add MissingContainer and LockedTopic exception
Added by Felix Schäfer almost 3 years ago.
Updated over 2 years 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