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

SMTPUTF8 is required, but was not offered by host #4969

Closed
link2xt opened this issue Nov 9, 2023 · 2 comments · Fixed by #4984
Closed

SMTPUTF8 is required, but was not offered by host #4969

link2xt opened this issue Nov 9, 2023 · 2 comments · Fixed by #4984
Assignees
Labels
bug Something is not working

Comments

@link2xt
Copy link
Collaborator

link2xt commented Nov 9, 2023

We probably should base64/quoted-printable more when the message is not encrypted: https://support.delta.chat/t/smtputf8-is-required-but-was-not-offered-by-host/2788

The servers we are testing against (more or less default Postfix setup) support SMTPUTF8. mail.sangham.net (the sender server) also offers SMTPUTF8:

220 mail.sangham.net ESMTP Postfix (Ubuntu)
EHLO u
250-mail.sangham.net
250-PIPELINING
250-SIZE 10240000
250-ETRN
250-AUTH PLAIN LOGIN
250-ENHANCEDSTATUSCODES
250-8BITMIME
250-DSN
250 SMTPUTF8

Maybe this happens because the receiver does not support SMTPUTF8. In any case, we should probably assume it is not supported and simply encode all messages as if there is no SMTPUTF8 support anywhere.

For reference, SMTPUTF8 is specified in RFC 6531.

@link2xt link2xt added the bug Something is not working label Nov 9, 2023
@hpk42
Copy link
Contributor

hpk42 commented Nov 10, 2023 via email

@link2xt link2xt self-assigned this Nov 12, 2023
@link2xt
Copy link
Collaborator Author

link2xt commented Nov 12, 2023

SMTPUTF8 allows to put UTF-8 characters into email header values. I guess the error is caused by unencoded chat name in the List-ID:

format!("{} <{}>", chat.name, chat.grpid),

The code adding List-ID for broadcast lists was introduced recently in
8573649
(PR #4644)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is not working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants