Consider following From: header of a message From: Something

In <a class="issue-link js-issue-link" data-error-text="Failed to load title" data-id=

EmailAddress::__toString() fails if there the sender/recipient name contains a colon about imap HOT 4 CLOSED

ddeboer commented on May 20, 2024

EmailAddress::__toString() fails if there the sender/recipient name contains a colon

from imap.

Comments (4)

ddeboer commented on May 20, 2024

Thanks for the bug report. Are such From: headers valid? If so, would you like to open a PR to fix this?

from imap.

Slamdunk commented on May 20, 2024

Yes such From header is valid, but the spaces should be escaped.

@vmattila can you provide the original raw message headers?

from imap.

vmattila commented on May 20, 2024

I think I do not have the original message anymore to copy-paste the raw headers here.

As it has been a while since last update, I searched through the codebase and could not find imap_headerinfo() call anywhere. Before anybody gets confused as I was, must clarify that https://github.com/ddeboer/imap/blob/master/src/Message.php#L184 imap_header() is alias to imap_headerinfo() . ;)

from imap.

Slamdunk commented on May 20, 2024

In #199 we've put into test a standard complex email address:

imap/tests/fixtures/email_address.eml

Line 2 in aac8f58

Cc: "This one: is \"right\"" <[email protected]>, No-address

The format without the double-quotes is not valid and we can't do anything to fix it (except complex parsing of raw headers or throwing away the imap extension).
We have removed the __toString() function: use the getAddress() from now so at least you can catch handle graceful the error.

from imap.

Recommend Projects