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

Recipient header (To:) is not properly encoded #5475

Closed
MirLach opened this issue Oct 14, 2016 · 5 comments
Closed

Recipient header (To:) is not properly encoded #5475

MirLach opened this issue Oct 14, 2016 · 5 comments

Comments

@MirLach
Copy link

MirLach commented Oct 14, 2016

If recipient header contains some diacritics UTF-8 characters they are not sanitized - not converted to quoted string: =?UTF-8?Q?=
From and Subject is properly sanitized. I think it is related to some $mailto / $headers['To'] changes in recent version.

It results in this error and message is undeliverable:

Diagnostic-Code: X-Postfix; SMTPUTF8 is required, but was not offered by host
    mx1.seznam.cz[77.75.76.42]

Works in Roundcube 1.2.0. does not work in 1.2.2.

@alecpl
Copy link
Member

alecpl commented Oct 14, 2016

Works for me. Disable all plugins and try again. You can also enable smtp_debug to see what exactly is being sent to SMTP server.

@MirLach
Copy link
Author

MirLach commented Oct 14, 2016

I forget to note - we are using PHP mail() function, not SMTP connect.
Another instance with older 1.2.0 is using SMTP TLS connection.

On October 14, 2016 6:28:50 PM CEST, Aleksander Machniak [email protected] wrote:

Works for me. Disable all plugins and try again. You can also enable
smtp_debug to see what exactly is being sent to SMTP server.

You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub:
#5475 (comment)

Odeslano z maleho zarizeni. Prosim omluvte mou strucnost.

@alecpl
Copy link
Member

alecpl commented Oct 14, 2016

Ok, confirmed with mail(). I'll fix this for 1.2, but keep in mind mail() method will not be supported in 1.3.

@alecpl
Copy link
Member

alecpl commented Oct 14, 2016

Fixed.

@alecpl alecpl closed this as completed Oct 14, 2016
@MirLach
Copy link
Author

MirLach commented Oct 14, 2016

Thank you!

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

No branches or pull requests

2 participants