search

laminas / laminas-mime

Create and parse MIME messages and parts
https://docs.laminas.dev/laminas-mime/
BSD 3-Clause "New" or "Revised" License
29 stars 23 forks source link

A logical bug occurs while splitting header field #2

Closed weierophinney closed 3 years ago

weierophinney commented 4 years ago

According to https://tools.ietf.org/html/rfc5322, the following email address example is completely ok:

"Some Name" <email@address.com>

But when you pass this value to Zend\Mime\Decoder::splitHeaderField(); it returns a wrong value because of a logical bug in the code (Line #181, the condition is not correct for all cases):

Expected results

"Some Name" <email@address.com>

Actual results

"Some Name" <email@address.com

So, in the actual result, the last character is missing.

I would like to create a merge request with valid tests if you think this should be fixed.

Originally posted by @ozanakman at https://github.com/zendframework/zend-mime/issues/40

weierophinney commented 4 years ago

@ozanakman sure, please submit PR. Thanks!

Originally posted by @michalbundyra at https://github.com/zendframework/zend-mime/issues/40#issuecomment-549858385

weierophinney commented 3 years ago

This package is considered feature-complete, and is now in security-only maintenance mode, following a decision by the Technical Steering Committee. If you have a security issue, please follow our security reporting guidelines. If you wish to take on the role of maintainer, please nominate yourself

If you are looking for an actively maintained package alternative, we recommend: