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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

bkonetzny’s picture

bkonetzny CreditAttribution: 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 CreditAttribution: d.sibaud commented
Status: Needs review » Reviewed & tested by the community

Working patch!

roball’s picture

roball CreditAttribution: roball commented
Priority: Normal » Critical
FileSize
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 CreditAttribution: 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 CreditAttribution: Valeratal commented

Subscribe

tallsimon’s picture

tallsimon CreditAttribution: tallsimon commented

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

yang_yi_cn’s picture

yang_yi_cn CreditAttribution: yang_yi_cn commented

interesting, subscribe

drupalninja99’s picture

drupalninja99 CreditAttribution: drupalninja99 commented

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

grey_’s picture

grey_ CreditAttribution: 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 CreditAttribution: dkruglyak commented

Can we get this patch finally committed? It works.

univate’s picture

univate CreditAttribution: univate commented

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

+1 RTBC

roball’s picture

roball CreditAttribution: roball commented

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

franz’s picture

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

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

roball’s picture

roball CreditAttribution: 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 CreditAttribution: 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 CreditAttribution: MrAdamJohn commented

@Nareshchowdary you should start a new thread..

zhongguo999999’s picture

zhongguo999999 CreditAttribution: 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?