With using SMTP module the embedded images are there but don't show in the e-mails. After some debugging, it seems the multipart/related content-type misses the boundary delimiter so the images can't be found. The attached patch fixes that.

CommentFileSizeAuthor
#3 smtp+mimemail.zip2.59 KBroball
smtp.boundary_01.patch984 bytessgabe

Comments

bkonetzny’s picture

bkonetzny commented

I can confirm this patch works as described.
Inline images, sent from mimemail, are no longer missing.
Thanks!

d.sibaud’s picture

d.sibaud commented
Status: Needs review » Reviewed & tested by the community

Working patch!

roball’s picture

roball commented
Priority: Normal » Critical
StatusFileSize
new smtp+mimemail.zip2.59 KB

Changing this bug to critical, since it completely breaks compatibility with the recently released Mime Mail Module 6.x-1.0-alpha3. I have been using the SMTP Authentication Support Module 6.x-1.0-beta4 together with Mime Mail versions < 6.x-1.0-alpha3 without any problems for many months, but the recent update to Mime Mail 6.x-1.0-alpha3 no longer led to readable received mails. Fortunately, sgabe (a new co-maintainer of Mime Mail) unveiled the problem being caused by SMTP. I applied his patch attached above and I can now confirm that it indeed solved the problem.

FYI, I have attached the resulting mbox files of sending out the same HTML mail (which does not even have an attachment - it's just dual HTML & plain-text) using Mime Mail 6.x-1.0-alpha3 with the original SMTP 6.x-1.0-beta4 ("mimemail-alpha3_smtp-beta4_FAILED.mbox") - which failed - and after applying the patch ("mimemail-alpha3_smtp-beta4+patch727744_WORKS.mbox") - which worked.

So please commit this patch asap - thanks!

sgabe’s picture

sgabe commented

@roball: Just to let you know, if you press F9 and take a look at the message source in The Bat! the message body is there in the e-mail, but without proper boundary delimiter the e-mail clients can't interpret the whole message. I guess The Bat! is even skipping that part on exporting.

Valeratal’s picture

Valeratal commented

Subscribe

tallsimon’s picture

tallsimon commented

You are a genius! Saved me a major headache. This patch works well.

yang_yi_cn’s picture

yang_yi_cn commented

interesting, subscribe

drupalninja99’s picture

drupalninja99 commented

ive got that working but attachments don't show up.

grey_’s picture

grey_ commented

Thanks!

Simplenews 6.x-2.x-dev and Mimemail 6.x-1.0-alpha4 was broken until this fix.

åäö characters was completely garbled.

dkruglyak’s picture

dkruglyak commented

Can we get this patch finally committed? It works.

univate’s picture

univate commented

Can confirm the same bug and that the patch in the original post fixes the issue

+1 RTBC

roball’s picture

roball commented

Is there any reason why this essential patch does not get committed?

franz’s picture

franz
any
Location Montréal
commented
Status: Reviewed & tested by the community » Fixed

Sorry for the time it took, comitted to dev version...

roball’s picture

roball commented

Thank you for committing! Finally, we have a release of SMTP Authentication Support (6.x-1.0-beta5) that is working fine with the latest release of Mime Mail (6.x-1.0-alpha4). Just tested it successfully.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Nareshchowdary’s picture

Nareshchowdary commented

When I use SMTP with Mimemail attachments are not going..
I m using smtp 6.x-1.0-beta5 and Mimemail 6.x-1.0-alpha6.
Please help on this..
Regards,
Naresh

mradamjohn’s picture

mradamjohn commented

@Nareshchowdary you should start a new thread..

zhongguo999999’s picture

zhongguo999999 commented

I found a same bug in Contact Attach module.

If it is a bug in SMTP Authentication Support, why don't fix it? other reasons?