Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Failing reference parsing #1341

Closed
wants to merge 1 commit into from
Closed

Failing reference parsing #1341

wants to merge 1 commit into from

Conversation

ylecuyer
Copy link

Hi,

We have noticed lately a parse error on the Reference header.
We never had trouble until we reached this one, but when I looked inside our db this issue is present for a lot of emails.

I have attached to this PR two test with the parsing failure. One is when references are separated with new line and the other is when the references are separated with comma

@ylecuyer
Copy link
Author

@jeremy I'm sorry I made you loose time yesterday. I brainfucked and now I understand why.
We couldn't parse this kind of References header because we use 2.7.1 and not master.

I'm trying to backport a83fcfa into the 2.7.2 version to be released but this is not straightforward.

I tried to update to 2.8 using master branch but it is impossible because of this bug : #1299 which seems to have been fixed here #1300 but not merged

Eventually I think I'll try to upgrade using the mater branch using the commit just before 404ecdd this should be the easiest way to have this all working

@ylecuyer ylecuyer closed this May 29, 2019
@ylecuyer ylecuyer deleted the failing-reference-parsing branch May 29, 2019 13:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants