Patch #17705
MailHandler should ignore bogus issue strings [some-string#1234] in subject
| Status: | Closed | Start date: | ||
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: |  Felix Schäfer | % Done: | 0% | |
| Category: | Email receiving | |||
| Target version: | 2.6.1 |
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.
History
#1
Updated by Jean-Baptiste Barth over 6 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!
#2
Updated by Felix Schäfer over 6 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#dispatchtoo.
I'll see about that, but I guess it's out of scope for this ticket.
#4
Updated by Felix Schäfer over 6 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.
#5
Updated by Felix Schäfer over 6 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.
#6
Updated by Frank Paaske about 6 years ago
Is this planned to be included in a release soon? I think this would make the email integration with OTRS much simpler!
#7
Updated by Jean-Philippe Lang about 6 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.