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

Cyrus IMAP BINARY FETCH error handling #9097

Closed
alecpl opened this issue Aug 13, 2023 · 1 comment
Closed

Cyrus IMAP BINARY FETCH error handling #9097

alecpl opened this issue Aug 13, 2023 · 1 comment
Milestone

Comments

@alecpl
Copy link
Member

alecpl commented Aug 13, 2023

When Cyrus can't decode a mail part body requested in BINARY FETCH command it will not return an error (NO response) as you'd expect. Therefore Roundcube does not fallback to non-binary FETCH.

However, it does not return BINARY item in the response either, so we should be able to detect the error.

C: A0008 UID FETCH 2 (BINARY.PEEK[2])
S: * 2 FETCH (UID 2)
S: A0008 OK Completed (0.000 sec)

Reference: cyrusimap/cyrus-imapd#4567

@alecpl alecpl added this to the 1.6.3 milestone Aug 13, 2023
alecpl added a commit that referenced this issue Aug 13, 2023
alecpl added a commit that referenced this issue Aug 13, 2023
@alecpl
Copy link
Member Author

alecpl commented Aug 13, 2023

Fixed.

@alecpl alecpl closed this as completed Aug 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant