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

Send IMAP ID only after authentication #7517

Closed
wants to merge 1 commit into from
Closed

Send IMAP ID only after authentication #7517

wants to merge 1 commit into from

Conversation

ledgr
Copy link
Contributor

@ledgr ledgr commented Jul 29, 2020

In case of IMAP proxy, that is doing authentication and routing, receiving ID command may result in BAD Invalid command error, as IMAP proxy does not yet know which server should receive the communications. Even if proxies would return correct output for ID command, the "name" field returned would not necessarily be the same as for "name" field value after authenticating.

This applies to Nginx + Dovecot stack.

iPhone Mail, Mac OS X Mail, Microsoft Outlook, Thunderbird, and other major (as 30+ minor) MUAs are sending ID command after authenticating, or even delay it until after NAMESPACE.

@alecpl
Copy link
Member

alecpl commented Jul 31, 2020

I get the idea, but there's a plugin that sets ident property in storage_connect hook. We should keep that possibility.

@alecpl
Copy link
Member

alecpl commented Jan 24, 2021

Implemented in 5975742.

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

Successfully merging this pull request may close these issues.

2 participants