Vincent Dauce comments

Results 51 comments of


                                            Vincent Dauce

Default fallback for mail clients that don't set content-name

You can find another fork that already fix all cases : https://github.com/eXorus/php-mime-mail-parser

Attachment class __construct arg need tweak

I guess it's only for a specific email. Could you provide the raw version of the email to investigate because it's weird to have no contentId for an attachment?

Update MailCare Request

Yes still an issue, but the PR are not merged. I'm not sure Laravel mailbox is still used.

[Docs] How to create a driver ?

Thanks a lot I gonna try.

[Docs] How to create a driver ?

It will be nice to add the steps in the docs

Parser::fromStream(): do not close stream

Thanks, could you add a test to show the issue, because for now before and after your change all the tests are green. Are we agree that this issue is...

Memory leak parsing .eml in a loop

I reproduced the issue but I don't think I can do something it's a bug on mailparse as you said in another issue. https://github.com/php-mime-mail-parser/php-mime-mail-parser/issues/344 I could investigate because I have...

Last header is ignored when email has no body

Are you sure that an email without a body is right? I guess it's not a valid email and all emails should be: header1 header2 --blank line-- --body--

Last header is ignored when email has no body

This one is more complexe to handle because the issue is on mailparse, not php-mime-mail-parser. With 2 lines the mailparse structure is: ``` array(11) { ["headers"]=> array(2) { ["from"]=> string(17)...

Last header is ignored when email has no body

I don't know if there is an issue for this, you could check here: https://bugs.php.net/search.php?cmd=display&package_name[]=mailparse It doesn't change anything with a content-type: ``` Content-type: text/plain From: Date: Mon, 17 Sep...