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

Enable UTF8=ACCEPT if the server supports ENABLE (and UTF=ACCEPT). #1617

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arnt
Copy link

@arnt arnt commented Sep 12, 2024

This is written so that net-imap has the option of enabling IMAP4REV2 instead, if the IMAP server has that. (IMAP4Rev2 includes the same syntax and functionality.)

With this PR, mikel-mail is able to read a message with UTF8 addresses from a compliant IMAP server, generate a reply and send it. Both net-imap and net-smtp contain the necessary code. At one line of code, this is the smallest such PR I've ever made ;)

(You may wonder about the call to .capable?. It's enough, see RFF5161 page 2. ENABLE is written to allow clients to send a fixed string; the server ignores what it can't enable.)

This is written so that net-imap has the option of enabling IMAP4REV2 instead, if available.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant