Project

General

Profile

Actions

Patch #16190

closed

Relax rejections based on Auto-Submitted header

Added by Felix Schäfer over 10 years ago. Updated over 9 years ago.

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

0%

Estimated time:

Description

One of our (Plan.io) uses an email server to forward messages to a Redmine instance which inserts Auto-Submitted: auto-forwarded headers to the forwarded emails. Those emails are then rejected because of source:/trunk/app/models/mail_handler.rb@12937#L69.

Putting the fact aside that this header is only part of an IETF draft and not of an RFC, and that even according to that draft it shouldn't be there, this header causes the forwarded emails to be reject. We propose changing the Regex to only match the undesirable values defined in RFC3834, i.e. /^auto-(replied|generated)/.


Files

16190.patch (678 Bytes) 16190.patch Felix Schäfer, 2014-02-27 12:57

Related issues

Related to Redmine - Feature #11338: Exclude emails with auto-submitted => auto-generatedClosedJean-Philippe Lang

Actions
Related to Redmine - Defect #15999: Ignoring out of office emailsClosed

Actions
Related to Redmine - Defect #20465: MailHandler: ignoring email with Auto-Submitted:auto-generated headerNeeds feedback

Actions
Actions #1

Updated by Felix Schäfer over 10 years ago

And here is the patch.

Actions #2

Updated by Holger Just over 10 years ago

To cite from RFC 3834

Recipients of messages containing an Auto-Submitted field with any keyword other than "no" MAY assume that the message was not manually submitted by a human.

Thus, although it may be correct to swallow this email, it makes it much harder for people using kind-of-broken mail servers (like Lotus Domino) which forcefully include the Auto-Submitted: auto-forwarded header into all outgoing mails touched by their forwarding script.

In addition, given that forwarded mails generally are assumed to be valid, we could thus argue that the auto-forwarded mails are not to be dropped.

Actions #3

Updated by Markus Plutka about 10 years ago

+1

Actions #4

Updated by Jan Niggemann (redmine.org team member) about 10 years ago

Patch verified (and it's trivial anyway), can someone please merge this?

IMHO it's important that we merge such trivial patches right away and not let the contributor wait too long...

Actions #5

Updated by Miquel Coll almost 10 years ago

Jan Niggemann (redmine.org team member) wrote:

Patch verified (and it's trivial anyway), can someone please merge this?

IMHO it's important that we merge such trivial patches right away and not let the contributor wait too long...

I tried the patch in redmine-2.4.5 and worked fine. i'm using Lotus Domino and now the messages are not rejected.

Actions #6

Updated by Toshi MARUYAMA almost 10 years ago

  • Target version set to 2.6.0
Actions #7

Updated by Toshi MARUYAMA almost 10 years ago

  • Related to Feature #11338: Exclude emails with auto-submitted => auto-generated added
Actions #8

Updated by Toshi MARUYAMA almost 10 years ago

Actions #9

Updated by Toshi MARUYAMA almost 10 years ago

  • Status changed from New to Closed

Committed in trunk r13205, r13206 and r13207, thanks.

Actions #10

Updated by Toshi MARUYAMA over 8 years ago

  • Related to Defect #20465: MailHandler: ignoring email with Auto-Submitted:auto-generated header added
Actions

Also available in: Atom PDF