Project

General

Profile

Actions

Feature #38273

closed

Improve errors in MailHandler: add MissingContainer and LockedTopic exception

Added by Felix Schäfer about 1 year ago. Updated about 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Email receiving
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed

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

1-missing_container_error.patch (1.92 KB) 1-missing_container_error.patch Felix Schäfer, 2023-02-13 15:40
2-locked_topic_error.patch (859 Bytes) 2-locked_topic_error.patch Felix Schäfer, 2023-02-13 15:40

Related issues

Blocks Redmine - Feature #38274: Receive e-mail replies to news and news commentsClosedGo MAEDA

Actions
Blocked by Redmine - Feature #38263: Try importing journal replies as issue reply where applicableClosedGo MAEDA

Actions
Actions #1

Updated by Felix Schäfer about 1 year ago

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.

Actions #2

Updated by Holger Just about 1 year ago

  • Blocks Feature #38274: Receive e-mail replies to news and news comments added
Actions #3

Updated by Go MAEDA about 1 year ago

  • Target version set to 5.1.0

Setting the target version to 5.1.0.

Actions #4

Updated by Go MAEDA about 1 year ago

  • File 1-missing_container_error-v2.patch added

Updated the first patch to apply cleanly to the current trunk.

Actions #5

Updated by Go MAEDA about 1 year ago

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.

Actions #6

Updated by Go MAEDA about 1 year ago

  • File deleted (1-missing_container_error-v2.patch)
Actions #7

Updated by Go MAEDA about 1 year ago

  • Blocked by Feature #38263: Try importing journal replies as issue reply where applicable added
Actions #8

Updated by Go MAEDA about 1 year ago

  • 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.

Actions

Also available in: Atom PDF