Project

General

Profile

Bug #7146

IMAP: messages with a mime-attachment of type "message/delivery-notification" fail

Added by tonyarkles (Tony Arkles) over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Normal
Assignee:
shugo (Shugo Maeda)
Target version:
-
ruby -v:
Backport:
2.3: UNKNOWN, 2.4: UNKNOWN, 2.5: UNKNOWN
[ruby-core:47920]

Description

When doing im.fetch("1:*", "FULL") on my inbox, I discovered an email message that had a delivery notification attached to it. This failed to parse around line 2414 in imap.rb. I've fixed this in https://github.com/ruby/ruby/pull/193 and https://github.com/ruby/ruby/pull/194.

Associated revisions

Revision bd6bb6c0
Added by shugo (Shugo Maeda) over 5 years 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/test_imap_response_parser.rb: related test.

git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@37240 b2dd03c8-39d4-4d8f-98ff-823fe69b080e

Revision 37240
Added by shugo (Shugo Maeda) over 5 years 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/test_imap_response_parser.rb: related test.

Revision 37240
Added by shugo (Shugo Maeda) over 5 years 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/test_imap_response_parser.rb: related test.

Revision 37240
Added by shugo (Shugo Maeda) over 5 years 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/test_imap_response_parser.rb: related test.

Revision 37240
Added by shugo (Shugo Maeda) over 5 years 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/test_imap_response_parser.rb: related test.

Revision 37240
Added by shugo (Shugo Maeda) over 5 years 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/test_imap_response_parser.rb: related test.

History

#1 [ruby-core:47922] Updated by tonyarkles (Tony Arkles) over 5 years ago

Was in version "ruby 1.9.3p194" that I noticed the problem.

#2 [ruby-core:47925] Updated by shugo (Shugo Maeda) over 5 years ago

  • Status changed from Open to Assigned
  • Assignee changed from tonyarkles (Tony Arkles) to shugo (Shugo Maeda)

#3 [ruby-core:47955] Updated by tonyarkles (Tony Arkles) over 5 years ago

I accidentally broke the pull request. Fixed one at https://github.com/ruby/ruby/pull/196

#4 Updated by shugo (Shugo Maeda) over 5 years ago

  • Tracker changed from Feature to Bug

#5 Updated by shugo (Shugo Maeda) over 5 years ago

  • Status changed from Assigned 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/test_imap_response_parser.rb: related test.

Also available in: Atom PDF