Honour "quoted-printable" encoding

[infertux]

Honour "quoted-printable" encoding

[stromvirvel]

Okay, this was easier than I thought 😃

See:
https://docs.python.org/2/library/email.message.html?highlight=quoted%20printable#email.message.Message.get_payload

Optional decode is a flag indicating whether the payload should be decoded or not, according to the Content-Transfer-Encoding header.

I only had to set decode=true in get_payload(). Additionally, I cannot reproduce the same problem with a multipart message. I guess this is because of the different way you are getting the payload.

However, I could not find an according flag in python3 documentation:
https://docs.python.org/3/library/email.message.html

Can you test this code with py3 (I don't understand your python test file for now)?

[infertux]

Thanks a lot for the thorough writeup. Looking at the comment above, it seems we're deleting the header to fix a bug in Thunderbird. IIRC this used to crash Thunderbird a while ago. I would suggest to simply remove this line thereby keeping the header intact then check if Thunderbird has been fixed. Do you use Thunderbird by any chance?

[stromvirvel]

Simply leaving the header doesn't help aswell, as it applies to the encrypted text in the mail payload. Encrypted data never has umlauts in it. We really need to decode the quoted-printable text, re-encode to UTF8, and then encrypt.

I'm currently using Apple Mail :-)

[infertux]

Hmm okay I don't see why we need to decode the quoted-printable text though. Is this how Thunderbird and Apple Mail handle it? Sorry I can't investigate that right now.

On a side note, I'd like to avoid adding dependencies as much as possible, especially if it's not compatible with Python 3.

[stromvirvel]

I may be wrong, but if you simply encrypt a string =E4, how could gpg know that =E4 is encoded? The header only applies to the mail client, but not to gpg (AFAIK).

But I have to do further investigation. I'll test if simply leaving the header alive would actually help, and come back to you later.

[stromvirvel]

I can confirm, that leaving the header Content-Transfer-Encoding: quoted-printable doesn't fix this problem. It doesn't even help when I add the same header to the attached inline file "encrypted.asc".

I'll write a proof of concept (hopefully without adding dependencies ;-)) and come back to you later.