Do not supply charset parameter to set_payload. #48

Open
wants to merge 1 commit into
from

Projects

None yet

2 participants

Contributor

Specifying the charset to email.message.Message eventually instructs it to dismiss the charset specified in Content-Type and furthermore recode the payload charset according to the global mappings predefined in the module, which is totally unwanted.

@moriyoshi moriyoshi Do not supply charset parameter to set_payload.
It eventually instructs email.message.Message to dismiss the charset specified in Content-Type and furthermore recode the payload charset according to the global mappings predefined in the module, which is totally unwanted.
a5917da
Owner
mcdonc commented Aug 20, 2013

I'll need to swap this package back into my head to feel satisfied that this is a good change. That may take a bit.

Contributor

Later I noticed that the change like the following (see the latest two of https://github.com/moriyoshi/pyramid_mailer/commits/moriyoshi/give-charset-object-instead-of-name ) should make much more sense, but this change would be more acceptable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment