ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:99315] [Ruby master Bug#17049] Net::IMAP - Handling of NOOP untagged responses sent by Zimbra
@ 2020-07-24 19:53 christophe.fish
  2020-07-24 21:12 ` [ruby-core:99316] " merch-redmine
  0 siblings, 1 reply; 2+ messages in thread
From: christophe.fish @ 2020-07-24 19:53 UTC (permalink / raw)
  To: ruby-core

Issue #17049 has been reported by Nestorfish (Christophe Le Roy).

----------------------------------------
Bug #17049: Net::IMAP - Handling of NOOP untagged responses sent by Zimbra
https://bugs.ruby-lang.org/issues/17049

* Author: Nestorfish (Christophe Le Roy)
* Status: Open
* Priority: Normal
* ruby -v: ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-darwin17]
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN
----------------------------------------
Zimbra server sends invalid untagged responses to prevent some clients from disconnecting during long-running requests.
As they are invalid, they raise an exception in Net::IMAP, while they could be clearly identified and safely ignored.

I have opened an issue on net-imap GitHub repository, along with a pull request (https://github.com/ruby/net-imap/issues/2 and https://github.com/ruby/net-imap/pull/3), but it seems I should have reported the issue here.

Can we have a look to these ?



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 2+ messages in thread

* [ruby-core:99316] [Ruby master Bug#17049] Net::IMAP - Handling of NOOP untagged responses sent by Zimbra
  2020-07-24 19:53 [ruby-core:99315] [Ruby master Bug#17049] Net::IMAP - Handling of NOOP untagged responses sent by Zimbra christophe.fish
@ 2020-07-24 21:12 ` merch-redmine
  0 siblings, 0 replies; 2+ messages in thread
From: merch-redmine @ 2020-07-24 21:12 UTC (permalink / raw)
  To: ruby-core

Issue #17049 has been updated by jeremyevans0 (Jeremy Evans).

Assignee set to shugo (Shugo Maeda)

Per RFC 3501 Section 7: `The client MUST be prepared to accept any response at all times.`  Arguably, raising an exception is not proper preparation.  Looking at the formal grammar, it does not appear that `* NOOP` is a valid server response (though `* NO OP` would be), so Zimbra's behavior does appear to go against the standard.  I think if we were going to fix this in net/imap, we should not make this specific to `* NOOP`, we should handle any unrecognized untagged response the same way.  However, @shugo is the maintainer of net/imap, so the behavior is up to him.

It looks like Zimbra's bug tracker for the IMAP component is: https://bugzilla.zimbra.com/buglist.cgi?component=IMAP%2FPOP%20Server&product=ZCS&resolution=---  I recommend you report the issue there.  You could also create a patch and add it as a pull request to https://github.com/Zimbra/zm-mailbox/pulls.  However, it appears the behavior in Zimbra is deliberate to keep connections active, so you may encounter some resistance.  According to RFC 3501, sending NOOP to keeping connections open is the job of the client (NOOP requests), it's not the job of the server.

----------------------------------------
Bug #17049: Net::IMAP - Handling of NOOP untagged responses sent by Zimbra
https://bugs.ruby-lang.org/issues/17049#change-86711

* Author: Nestorfish (Christophe Le Roy)
* Status: Open
* Priority: Normal
* Assignee: shugo (Shugo Maeda)
* ruby -v: ruby 2.6.5p114 (2019-10-01 revision 67812) [x86_64-darwin17]
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN
----------------------------------------
Zimbra server sends invalid untagged responses to prevent some clients from disconnecting during long-running requests.
As they are invalid, they raise an exception in Net::IMAP, while they could be clearly identified and safely ignored.

I have opened an issue on net-imap GitHub repository, along with a pull request (https://github.com/ruby/net-imap/issues/2 and https://github.com/ruby/net-imap/pull/3), but it seems I should have reported the issue here.

Can we have a look to these ?



-- 
https://bugs.ruby-lang.org/

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2020-07-24 21:12 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-07-24 19:53 [ruby-core:99315] [Ruby master Bug#17049] Net::IMAP - Handling of NOOP untagged responses sent by Zimbra christophe.fish
2020-07-24 21:12 ` [ruby-core:99316] " merch-redmine

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).