-
-
Notifications
You must be signed in to change notification settings - Fork 172
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
content id for attachments #91
Comments
Is there a workarond to add contend-id header for attachment? |
msg = Message(.....) but you should use my bug fix from master branch e195fca |
One who don't have bugfix e195fca can pass headers as a list I guess:
|
Confirmed, simply adding .items() fixes it |
Looks like the workaround makes some problems when opening the Mail with (at least) Thunderbird. The CID image is well attached but Thunderbird has problems displaying it in the html corpus. When the message is a |
Same problem with Thunderbird here. I edited the following in |
I think it also would work if the images would be attached to the |
need to set content id for attachments to be able to embed images using cid method
also the line 373 should be changed to:
for key, value in attachment.headers.items():
need the .items():
The text was updated successfully, but these errors were encountered: