Skip to content

Conversation

@frioux
Copy link

@frioux frioux commented Mar 19, 2016

No description provided.

@rjbs
Copy link
Collaborator

rjbs commented Mar 25, 2016

I wonder why it parses the headers, instead of just using them.

Anyway, seems reasonable, maybe. Maybe should use Email::MessageID instead.

Want some comaint?

@frioux
Copy link
Author

frioux commented Mar 25, 2016

You would know better than I would how to deal with the headers, so I defer
to you in those matters.

And no I don't want comaint.

sent from a rotary phone, pardon my brevity
On Mar 24, 2016 6:25 PM, "Ricardo Signes" [email protected] wrote:

I wonder why it parses the headers, instead of just using them.

Anyway, seems reasonable, maybe. Maybe should use Email::MessageID instead.

Want some comaint?


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#5 (comment)

@pali
Copy link

pali commented Feb 12, 2018

I wonder why it parses the headers, instead of just using them.

It is wrong to try parsing Message-Id headers with parser for addr-list structures. It is even worse to try parsing them via Email::Address module.

And yes, you can just return those Message-Id headers as you are not interested in its content. I fixes it in pull request: #6

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.

3 participants