search

uvdesk / mailbox-component

The Mailbox component provides tools that allows your helpdesk to process tickets from multiple email sources.
MIT License
22 stars 38 forks source link

referenceIds header explode #70

Closed cdalpont-AMB closed 3 years ago

cdalpont-AMB commented 3 years ago

Some mail clients (ie OWA) use commas instead newline for multiple ids in "reference" header.

1. Why is this change necessary?

reference header is not correctly parsed when mail is sent from some clients (ie OWA). Some clients in fact use commas instead newline for separating multiple ids.

2. What does this change do, exactly?

The code looks if there is any comma in the reference header. If any, it use comma in the function "explode". There is non possibility that a message id contains commas, so if there is one comma it must be the separator. If comma is not found, the space is used as separator.

3. Please link to the relevant issues (if any).

papnoisanjeev commented 3 years ago

@cdalpont-AMB

Have you tested ticket create process after this update. Please verify as we are facing to add threads into ticket after this update (checked with gmail).

papnoisanjeev commented 3 years ago

@cdalpont-AMB

We are currently closing this pull request as it is creating issue for project normal flow. Please test again and specify your case with more steps.

You can again create a pull request for the same after correction in your code.