Project

General

Profile

Actions

Patch #17705

closed

MailHandler should ignore bogus issue strings [some-string#1234] in subject

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

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

0%

Estimated time:

Description

Currently the mail handler will try to parse anything that remotely resembles square brackets containing a hash followed by a number, whatever else happens to be in front of the hash in the brackets. This leads the mail handler to try to parse things like:

[12345678901234567890#1234]

and

[some-string#1234]
used by other systems.

The proposed patch Requires the hash to be either preceded by the opening bracket [ or a space.


Files

Actions #1

Updated by Jean-Baptiste Barth over 9 years ago

No problem merging that as long as it has some more tests for failing strings we introduce... and for good ones too, as I can't find tests in test/unit/mail_handler_test.rb. Maybe it's time to refactor the logic in MailHandler#dispatch too. Felix: can you handle that ? Thanks!

Actions #2

Updated by Felix Schäfer over 9 years ago

Jean-Baptiste Barth wrote:

No problem merging that as long as it has some more tests for failing strings we introduce... and for good ones too, as I can't find tests in test/unit/mail_handler_test.rb.

Sure, can you assign it back to me so that I can remember it? Thanks :-)

Maybe it's time to refactor the logic in MailHandler#dispatch too.

I'll see about that, but I guess it's out of scope for this ticket.

Actions #3

Updated by Jean-Baptiste Barth over 9 years ago

  • Assignee set to Felix Schäfer

Here we go!

Actions #4

Updated by Felix Schäfer over 9 years ago

Jean-Baptiste, there's already tests for good strings of sorts with the test that try to import emails and make sure they've been imported correctly. I fear the best I can do would be to provide 1 or 2 examples of incoming emails that should fail, or even with bad and good strings pointing to different issues.

Actions #5

Updated by Felix Schäfer over 9 years ago

Here's a patch with 2 test-emails containing subjects with strings we don't want to match anymore pointing to the wrong issue and strings we want to match pointing to the right one.

Actions #6

Updated by Frank Paaske over 9 years ago

Is this planned to be included in a release soon? I think this would make the email integration with OTRS much simpler!

Actions #7

Updated by Jean-Philippe Lang over 9 years ago

  • Subject changed from Improve incoming email issue subject recognition to MailHandler should ignore bogus issue strings [some-string#1234] in subject
  • Status changed from New to Resolved
  • Target version set to 2.6.1

Fix committed in r13593 with tests edited, thanks.

Actions #8

Updated by Jean-Philippe Lang over 9 years ago

  • Status changed from Resolved to Closed

Merged.

Actions

Also available in: Atom PDF