[bug]: SIEVE LOGOUT should respond with OK, not BYE about imap-server

[the-djmaze]

I am correct. BYE is only valid for other cases.
See https://datatracker.ietf.org/doc/html/rfc5804#section-1.2

This response can be issued at any time and should be immediately followed by a server hang-up of the connection

[mdecimus]

That is incorrect, according to the ManageSieve grammar, both OK, NO and BYE are valid responses for the LOGOUT command:

    response-logout       = response-oknobye
    response-oknobye    = response-ok / response-nobye
    response-nobye        = ("NO" / "BYE") [SP "(" resp-code ")"]
                            [SP string] CRLF
                            ;; The string contains human-readable text
                            ;; encoded as UTF-8.

[stappersg]

And later there was commit d94a6a2ec6d5f3.
I did learn that from the-djmaze/snappymail#1455 (comment)