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

Skip LOGIN attempt if LOGINDISABLED is set #322

Merged

Conversation

atc0005
Copy link
Owner

@atc0005 atc0005 commented Nov 18, 2022

In short, do not attempt to login if the IMAP server has indicated that logins are disabled for the current connection (state).

CHANGES:

  • doc comment tweaks
  • warn if TLS is not enabled when performing login attempt
  • early exit if unable to detect whether the LOGINDISABLED capability is set by the IMAP server
  • early exit if LOGINDISABLED capability is confirmed to be set by the IMAP server

REFERENCES:

In short, do not attempt to login if the IMAP server has
indicated that logins are disabled for the current
connection (state).

CHANGES:

- doc comment tweaks
- warn if TLS is not enabled when performing login attempt
- early exit if unable to detect whether the `LOGINDISABLED`
  capability is set by the IMAP server
- early exit if `LOGINDISABLED` capability is confirmed to be
  set by the IMAP server

REFERENCES:

- refs GH-321
- https://datatracker.ietf.org/doc/html/rfc3501#section-6.1.1
- https://datatracker.ietf.org/doc/html/rfc3501#section-6.2.3
@atc0005 atc0005 added this to the Next Patch Release milestone Nov 18, 2022
@atc0005 atc0005 self-assigned this Nov 18, 2022
@atc0005 atc0005 merged commit bc0d7d1 into master Nov 18, 2022
@atc0005 atc0005 deleted the i321-skip-login-imap-cmd-attempt-if-logindisabled-advertised branch November 18, 2022 11:59
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.

Skip use of LOGIN IMAP command if LOGINDISABLED capability is advertised
1 participant