Regex too greedy on multiple encoded names in comma separated list

[Knyffen]

Hello

When a comma separated list contains multiple names are encoded, the consumePhrase function in Mail::Message::Field::Full contains a greedy regex that spans across multiple email addresses.
An example is "=?utf-8?q?foo?= bar@mail.com, =?utf-8?q?baz?= qux@mail.com" where consumePhrase greedily selects the entirety of "=?utf-8?q?foo?= bar@mail.com, =?utf-8?q?baz?=" as the phrase.

I believe that the issue can be fixed by replacing
elsif($string =~ s/^\s*((?:\=\?.*\?\=|[${atext}${atext_ill}\ \t.])+)//o )
with
elsif($string =~ s/^\s*((?:\=\?.*?\?\=|[${atext}${atext_ill}\ \t.])+)//o )
on

perl5-Mail-Message/lib/Mail/Message/Field/Full.pm

Line 643 in a49dad0

elsif($string =~ s/^\s*((?:\=\?.*\?\=|[${atext}${atext_ill}\ \t.])+)//o )

to make it non-greedy.

Best regards
Jonas

[markov2]

I do agree that adding the '?' is needed. Thank you, Jonas!

The fix is on github. Usually, I create new releases when I have collected a few.

[Knyffen]

Great! Thank you for the quick response!
Have a nice day :-)

[Knyffen]

Oh, and my name is Jonas Ryssel. Knyffen is my alias on the internet, but either is fine. Sorry for the confusion (I didn't expect to get my name in there). :-)