Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
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.
Comment | File | Size | Author |
---|---|---|---|
#3 | smtp+mimemail.zip | 2.59 KB | roball |
smtp.boundary_01.patch | 984 bytes | sgabe | |
Comments
Comment #1
bkonetzny CreditAttribution: bkonetzny commentedI can confirm this patch works as described.
Inline images, sent from mimemail, are no longer missing.
Thanks!
Comment #2
d.sibaud CreditAttribution: d.sibaud commentedWorking patch!
Comment #3
roball CreditAttribution: roball commentedChanging 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!
Comment #4
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.
Comment #5
Valeratal CreditAttribution: Valeratal commentedSubscribe
Comment #6
tallsimon CreditAttribution: tallsimon commentedYou are a genius! Saved me a major headache. This patch works well.
Comment #7
yang_yi_cn CreditAttribution: yang_yi_cn commentedinteresting, subscribe
Comment #8
drupalninja99 CreditAttribution: drupalninja99 commentedive got that working but attachments don't show up.
Comment #9
grey_ CreditAttribution: grey_ commentedThanks!
Simplenews 6.x-2.x-dev and Mimemail 6.x-1.0-alpha4 was broken until this fix.
åäö characters was completely garbled.
Comment #10
dkruglyak CreditAttribution: dkruglyak commentedCan we get this patch finally committed? It works.
Comment #11
univate CreditAttribution: univate commentedCan confirm the same bug and that the patch in the original post fixes the issue
+1 RTBC
Comment #12
roball CreditAttribution: roball commentedIs there any reason why this essential patch does not get committed?
Comment #13
franzSorry for the time it took, comitted to dev version...
Comment #14
roball CreditAttribution: roball commentedThank 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.
Comment #16
Nareshchowdary CreditAttribution: Nareshchowdary commentedWhen 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
Comment #17
MrAdamJohn CreditAttribution: MrAdamJohn commented@Nareshchowdary you should start a new thread..
Comment #18
zhongguo999999 CreditAttribution: zhongguo999999 commentedI found a same bug in Contact Attach module.
If it is a bug in SMTP Authentication Support, why don't fix it? other reasons?