laminas-mime icon indicating copy to clipboard operation
laminas-mime copied to clipboard

Attachments with longs names are not being processed

Open Ralev93 opened this issue 2 years ago • 1 comments

Q A
Documentation no
Bugfix yes
BC Break no
New Feature no
RFC no
QA no

Bug Report

Q A
Version(s) 2.10.x

Summary

Attachments with filenames, longer than 76 char symbols, are not being processed.

Current behavior

Currently, Laminas\Mail successfully identifies long names and successfully splits them into multiple variables in \Laminas\Mail\Header\ContentDisposition::getFieldValue(). However, the multiple variables do not get recognized in Laminas\Mime\Decode::splitHeaderField() due to this check strcasecmp($name, $wantedPart) i.e. it compares "filename*0" with "filename"

How to reproduce

Try to process an email with an attachment with long name, for example "This_______________________________________________________is____________________long__________________________________name.txt"

Expected behavior

It should not compare with strcasecmp, but rather with a regex. Then if it is a multiple value, it should concatenate the values

Ralev93 avatar Jun 13 '22 15:06 Ralev93

@Ralev93 I think this should go against 2.9.x, as a bugfix.

Ocramius avatar Jun 13 '22 16:06 Ocramius