Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

multipart/alternative not handled correctly if more than 2 parts #33

Closed
leejo opened this issue Feb 21, 2017 · 1 comment
Closed

multipart/alternative not handled correctly if more than 2 parts #33

leejo opened this issue Feb 21, 2017 · 1 comment

Comments

@leejo
Copy link
Contributor

leejo commented Feb 21, 2017

The code here: https://github.com/rjbs/Email-Stuffer/blob/master/lib/Email/Stuffer.pm#L584 # tries to handle the multipart/alternative Content-Type so we don't get, say, both text and html versions of an e-mail being displayed at the same time. However, if you have a text/plain part, text/html part, and an attachment (or several) this fails completely and you end up with the text and html being displayed at the same time in the e-mail client because the nesting of the parts w/r/t to the Content-Type isn't correct and there is no multipart/alternative instruction in the headers to allow the client to DTRT.

@simongreen-net
Copy link

This is a duplicate of issue #25 and should be fixed as part of the pull request I submitted for that issue.

@leejo leejo closed this as completed Jun 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants