zendframework / zend-mail

Mail component from Zend Framework
BSD 3-Clause "New" or "Revised" License
96 stars 111 forks source link

Problems with mailadress parsing #127

Open Baztey opened 7 years ago

Baztey commented 7 years ago

Example Email-header: "Foo <bar" foo.bar@test.com

Description: The example email-header should be valid according to https://tools.ietf.org/html/rfc2822#section-3.4 but the function AdressList.php/addFromString matches it incorrect. The result has the following form: "bar <foo.bar@test.com" This is clearly not a valid adress and therefore causes exceptions in the following code

Solution: Changing the regex in the addFromString function from lazzy to greedy by removing "?" from the named capture group "name"

glensc commented 7 years ago

@Baztey created PR based on your comments, including tests: https://github.com/zendframework/zend-mail/pull/143

however, it needs more fixes, because encoding back creates invalid result.

btw, your bugreport here has lost <>, use markdown code blocks to preserve them.

michalbundyra commented 4 years ago

This repository has been closed and moved to laminas/laminas-mail; a new issue has been opened at https://github.com/laminas/laminas-mail/issues/45.