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

ImapProtocol: Add STATUS command support. #424

Merged
merged 1 commit into from
Jun 30, 2023
Merged

ImapProtocol: Add STATUS command support. #424

merged 1 commit into from
Jun 30, 2023

Conversation

InterLinked1
Copy link
Contributor

This is something I had to add to this library to make it work when I was playing around with it - hope it'll be useful
for other folks that run into the same issue:

This library currently contains some functions
that purport to get the status of a folder, but
these are misleading because they issue the
EXAMINE command, not the STATUS command,
and this is not actually considered "STATUS" in the IMAP protocol.

This adds support for STATUS at the protocol level and the folder level, and clarifies the difference in the documentation.

This library currently contains some functions
that purport to get the status of a folder, but
these are misleading because they issue the
EXAMINE command, not the STATUS command. This library
actually currently lacks support for the STATUS command,
and this command may be necessary to get some information
that the EXAMINE command will not provide.

This adds support for STATUS at the protocol level
and the folder level, and clarifies the difference
in the documentation.
@Webklex
Copy link
Owner

Webklex commented Jun 30, 2023

Hi @InterLinked1 ,
thanks a lot for your pr! It's highly appreciated :)

Should you discover another missing command, please feel welcome to add or report them as well.

Once again, thanks for taking the time and effort to make this library better!

Best regards and happy coding,

@Webklex Webklex merged commit 27d911c into Webklex:master Jun 30, 2023
@stevebauman stevebauman mentioned this pull request Jan 30, 2025
18 tasks
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.

2 participants