On Wed, Aug 23, 2017 at 3:02 AM, Matthieu Moy <git@xxxxxxxxxxxxxxx> wrote: >> On Tue, Aug 22, 2017 at 4:30 PM, Jacob Keller <jacob.keller@xxxxxxxxx> wrote: >>> Additionally I just discovered that the behavior here changes pretty >>> drastically if you have Email::Validate installed, now it splits the >>> address into multiple things: > > (I'm assuming you mean Email::Address, there's also Email::Valid but I > don't think it would modify the behavior) > No I actually definitely meant Email::Valid. I already had Mail::Address installed, and I then installed Email::Valid, and it changed behavior to split the cruft into multiple addresses. I don't actually know why or how it did this, but I'm certain it was presence of Email::Valid that did it. However, your first patch addresses the issue since you remove the cruft well before passing it into Email::Valid anyways. > Hmm, I think we reached the point where we should just stop using > Email::Address. I do agree, I don't think we should use Mail::Address. > > Patch series follows and should address both points. > > -- > Matthieu Moy > http://matthieu-moy.fr
