Bug #7153

IMAP: emails with (BODY ("MIXED")) returned from fetch fail to parse

Added by Tony Arkles over 1 year ago. Updated over 1 year ago.

[ruby-core:47951]
Status:Closed
Priority:Normal
Assignee:Shugo Maeda
Category:lib
Target version:2.0.0
ruby -v:1.9.3p194 Backport:

Description

I'm not sure if this is even valid IMAP (I couldn't find a reference to it in the RFC), but Google's IMAP server seems to return these occasionally. Found a message in my Inbox that wouldn't parse because of this.

I've added a test and fixed the module. Will post the pull request here right away.

Associated revisions

Revision 37240
Added by Shugo Maeda over 1 year ago

  • lib/net/imap.rb: fix Net::IMAP::ResponseParser to accept
    message/delivery-status ( [Bug #7146]),
    message/rfc822 attachments ( [Bug #7147]), and
    (BODY ("MIXED")) ( [Bug #7153]).

  • test/net/imap/testimapresponse_parser.rb: related test.

History

#1 Updated by Tony Arkles over 1 year ago

OK, the patch for this will have conflicts with the patches for #7145, #7146, and #7147 because it also adds unit tests to testimapresponse_parser.rb. I'm not sure what the best approach is for this... Whether to wait until that pull request is accepted and do a second one, or to just add this fix into the original pull request.

#3 Updated by Shugo Maeda over 1 year ago

  • Status changed from Open to Closed
  • % Done changed from 0 to 100

This issue was solved with changeset r37240.
Tony, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.


  • lib/net/imap.rb: fix Net::IMAP::ResponseParser to accept
    message/delivery-status ( [Bug #7146]),
    message/rfc822 attachments ( [Bug #7147]), and
    (BODY ("MIXED")) ( [Bug #7153]).

  • test/net/imap/testimapresponse_parser.rb: related test.

Also available in: Atom PDF