Scrambled accents in email body version 2.8.x #1477

Closed
swerter opened this Issue May 25, 2016 · 4 comments

Comments

Projects
None yet
2 participants
@swerter
Contributor

swerter commented May 25, 2016

Haraka version

2.8.0-2.8.4

Expected behavior

When sending an outgoing email from outlook 2010 with accents in it (eg àèü), the message body gets scrabled.

Observed behavior

Instead of á é í ó ú ñ, the email body shows � � � � � �

Message sent is multipart, with the text content part being:

------=_NextPart_000_002D_01D1B620.0C9A69F0`
Content-Type: text/plain;
charset="Windows-1252"
Content-Transfer-Encoding: base64

4SDpIO0g8yD6IPENCg0KClNlbnQgdmlhIE1pZ2FkdS5jb20sIHdvcmxkJ3MgZWFzaWVzdCBlbWFp
bCBob3N0aW5nCg==
------=_NextPart_000_002D_01D1B620.0C9A69F0
Content-Type: text/html;
charset="Windows-1252"
Content-Transfer-Encoding: quoted-printable

Steps to reproduce

Upgrade from haraka 2.7.3 to haraka 2.8

Please let me know if you need additional informations. Thanks!

@baudehlo

This comment has been minimized.

Show comment
Hide comment
@baudehlo

baudehlo May 25, 2016

Collaborator

I can't replicate this. Can you detail what plugins you are running?

Collaborator

baudehlo commented May 25, 2016

I can't replicate this. Can you detail what plugins you are running?

@swerter

This comment has been minimized.

Show comment
Hide comment
@swerter

swerter May 26, 2016

Contributor

I created a test repo: https://github.com/migadu/haraka_testcase_1477 with a readme how to replicate. It seems the set_banner method scrambles the html content.

Contributor

swerter commented May 26, 2016

I created a test repo: https://github.com/migadu/haraka_testcase_1477 with a readme how to replicate. It seems the set_banner method scrambles the html content.

@baudehlo

This comment has been minimized.

Show comment
Hide comment
@baudehlo

baudehlo May 26, 2016

Collaborator

Thanks for the test case - it helped massively.

Fixed in #1483

Collaborator

baudehlo commented May 26, 2016

Thanks for the test case - it helped massively.

Fixed in #1483

@baudehlo baudehlo closed this in #1483 May 27, 2016

@swerter

This comment has been minimized.

Show comment
Hide comment
@swerter

swerter May 27, 2016

Contributor

Most welcome! Thank you for Haraka.

Contributor

swerter commented May 27, 2016

Most welcome! Thank you for Haraka.

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