ruby-core@ruby-lang.org archive (unofficial mirror)
 help / color / mirror / Atom feed
* [ruby-core:100257] [Ruby master Bug#17207] message/disposition-notification: unexpected token LPAR (expected QUOTED or LITERAL)
@ 2020-10-01 13:41 jbecvar
  2020-11-24 22:26 ` [ruby-core:101058] " merch-redmine
  0 siblings, 1 reply; 2+ messages in thread
From: jbecvar @ 2020-10-01 13:41 UTC (permalink / raw)
  To: ruby-core

Issue #17207 has been reported by jbecvar (Jan Bečvář).

----------------------------------------
Bug #17207: message/disposition-notification: unexpected token LPAR (expected QUOTED or LITERAL)
https://bugs.ruby-lang.org/issues/17207

* Author: jbecvar (Jan Bečvář)
* Status: Open
* Priority: Normal
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN
----------------------------------------
With exception:
```
Net::IMAP::ResponseParseError: unexpected token LPAR (expected QUOTED or LITERAL)
    .../ruby/lib/net/imap.rb:3498:in `parse_error'
    .../ruby/lib/net/imap.rb:3350:in `match'
    .../ruby/lib/net/imap.rb:3298:in `case_insensitive_string'
    .../ruby/lib/net/imap.rb:2628:in `media_type'
    .../ruby/lib/net/imap.rb:2520:in `body_type_basic'
    .../ruby/lib/net/imap.rb:2515:in `body_type_1part'
    .../ruby/lib/net/imap.rb:2495:in `body'
    .../ruby/lib/net/imap.rb:2578:in `body_type_msg'
    .../ruby/lib/net/imap.rb:2509:in `body_type_1part'
    .../ruby/lib/net/imap.rb:2495:in `body'
    .../ruby/lib/net/imap.rb:2611:in `body_type_mpart'
    .../ruby/lib/net/imap.rb:2493:in `body'
    .../ruby/lib/net/imap.rb:2470:in `body_data'
    .../ruby/lib/net/imap.rb:2375:in `msg_att'
    .../ruby/lib/net/imap.rb:2345:in `numeric_response'
    .../ruby/lib/net/imap.rb:2287:in `response_untagged'
    .../ruby/lib/net/imap.rb:2258:in `response'
    .../ruby/lib/net/imap.rb:2184:in `parse'
```
While parsing the following (Cyrus 2.4.17):

```
C: RUBY0003 UID FETCH 3 (BODYSTRUCTURE)
S: * 1 FETCH (UID 3 BODYSTRUCTURE (("TEXT" "PLAIN" ("CHARSET" "UTF-8") NIL NIL "QUOTED-PRINTABLE" 439 9 NIL NIL NIL NIL)("MESSAGE" "DISPOSITION-NOTIFICATION" ("NAME" "MDNPart2.txt") NIL NIL "7BIT" 304 NIL ("INLINE" ("FILENAME" "MDNPart2.txt" "SIZE" "304")) NIL NIL) "REPORT" ("REPORT-TYPE" "disposition-notification" "BOUNDARY" "=_6568a7f155bf754f458c22bcc1da9919") NIL NIL NIL))
@str: "* 1 FETCH (UID 3 BODYSTRUCTURE ((\"TEXT\" \"PLAIN\" (\"CHARSET\" \"UTF-8\") NIL NIL \"QUOTED-PRINTABLE\" 439 9 NIL NIL NIL NIL)(\"MESSAGE\" \"DISPOSITION-NOTIFICATION\" (\"NAME\" \"MDNPart2.txt\") NIL NIL \"7BIT\" 304 NIL (\"INLINE\" (\"FILENAME\" \"MDNPart2.txt\" \"SIZE\" \"304\")) NIL NIL) \"REPORT\" (\"REPORT-TYPE\" \"disposition-notification\" \"BOUNDARY\" \"=_6568a7f155bf754f458c22bcc1da9919\") NIL NIL NIL))\r\n"
@pos: 213
@lex_state: EXPR_DATA
@token.symbol: LPAR
@token.value: "("
```

According to the RFC, IMAP::BodyTypeMessage should only be returned for message/rfc822, otherwise IMAP::BodyTypeBasic:
```
RFC 3501:
  A body type of type MESSAGE and subtype RFC822 contains, immediately after the basic fields,
  the envelope structure, body structure, and size in text lines of the encapsulated message.
  A body type of type TEXT contains, immediately after the basic fields, the size of the body
  in text lines... Extension data follows the basic fields and the type-specific fields listed above
...
    body-type-1part = (body-type-basic / body-type-msg / body-type-text) [SP body-ext-1part]
    body-type-basic = media-basic SP body-fields ; MESSAGE subtype MUST NOT be "RFC822"
    body-type-msg   = media-message SP body-fields SP envelope SP body SP body-fld-lines
    body-type-text  = media-text SP body-fields SP body-fld-lines
    body-ext-1part  = body-fld-md5 [SP body-fld-dsp [SP body-fld-lang [SP body-fld-loc *(SP body-extension)]]] ; MUST NOT be returned on non-extensible "BODY" fetch
    media-basic     = ((DQUOTE ("APPLICATION" / "AUDIO" / "IMAGE" / "MESSAGE" / "VIDEO") DQUOTE) / string) SP media-subtype
    media-message   = DQUOTE "MESSAGE" DQUOTE SP DQUOTE "RFC822" DQUOTE
```


---Files--------------------------------
imap.patch (3.34 KB)


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

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

* [ruby-core:101058] [Ruby master Bug#17207] message/disposition-notification: unexpected token LPAR (expected QUOTED or LITERAL)
  2020-10-01 13:41 [ruby-core:100257] [Ruby master Bug#17207] message/disposition-notification: unexpected token LPAR (expected QUOTED or LITERAL) jbecvar
@ 2020-11-24 22:26 ` merch-redmine
  0 siblings, 0 replies; 2+ messages in thread
From: merch-redmine @ 2020-11-24 22:26 UTC (permalink / raw)
  To: ruby-core

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


Can you please submit this as a pull request to the upstream repository: https://github.com/ruby/net-imap/pulls

----------------------------------------
Bug #17207: message/disposition-notification: unexpected token LPAR (expected QUOTED or LITERAL)
https://bugs.ruby-lang.org/issues/17207#change-88730

* Author: jbecvar (Jan Bečvář)
* Status: Open
* Priority: Normal
* ruby -v: ruby 3.0.0dev (2020-10-01T08:38:36Z master 2db081b5ff) [x86_64-linux]
* Backport: 2.5: UNKNOWN, 2.6: UNKNOWN, 2.7: UNKNOWN
----------------------------------------
With exception:
```
Net::IMAP::ResponseParseError: unexpected token LPAR (expected QUOTED or LITERAL)
    .../ruby/lib/net/imap.rb:3498:in `parse_error'
    .../ruby/lib/net/imap.rb:3350:in `match'
    .../ruby/lib/net/imap.rb:3298:in `case_insensitive_string'
    .../ruby/lib/net/imap.rb:2628:in `media_type'
    .../ruby/lib/net/imap.rb:2520:in `body_type_basic'
    .../ruby/lib/net/imap.rb:2515:in `body_type_1part'
    .../ruby/lib/net/imap.rb:2495:in `body'
    .../ruby/lib/net/imap.rb:2578:in `body_type_msg'
    .../ruby/lib/net/imap.rb:2509:in `body_type_1part'
    .../ruby/lib/net/imap.rb:2495:in `body'
    .../ruby/lib/net/imap.rb:2611:in `body_type_mpart'
    .../ruby/lib/net/imap.rb:2493:in `body'
    .../ruby/lib/net/imap.rb:2470:in `body_data'
    .../ruby/lib/net/imap.rb:2375:in `msg_att'
    .../ruby/lib/net/imap.rb:2345:in `numeric_response'
    .../ruby/lib/net/imap.rb:2287:in `response_untagged'
    .../ruby/lib/net/imap.rb:2258:in `response'
    .../ruby/lib/net/imap.rb:2184:in `parse'
```
While parsing the following (Cyrus 2.4.17):

```
C: RUBY0003 UID FETCH 3 (BODYSTRUCTURE)
S: * 1 FETCH (UID 3 BODYSTRUCTURE (("TEXT" "PLAIN" ("CHARSET" "UTF-8") NIL NIL "QUOTED-PRINTABLE" 439 9 NIL NIL NIL NIL)("MESSAGE" "DISPOSITION-NOTIFICATION" ("NAME" "MDNPart2.txt") NIL NIL "7BIT" 304 NIL ("INLINE" ("FILENAME" "MDNPart2.txt" "SIZE" "304")) NIL NIL) "REPORT" ("REPORT-TYPE" "disposition-notification" "BOUNDARY" "=_6568a7f155bf754f458c22bcc1da9919") NIL NIL NIL))
@str: "* 1 FETCH (UID 3 BODYSTRUCTURE ((\"TEXT\" \"PLAIN\" (\"CHARSET\" \"UTF-8\") NIL NIL \"QUOTED-PRINTABLE\" 439 9 NIL NIL NIL NIL)(\"MESSAGE\" \"DISPOSITION-NOTIFICATION\" (\"NAME\" \"MDNPart2.txt\") NIL NIL \"7BIT\" 304 NIL (\"INLINE\" (\"FILENAME\" \"MDNPart2.txt\" \"SIZE\" \"304\")) NIL NIL) \"REPORT\" (\"REPORT-TYPE\" \"disposition-notification\" \"BOUNDARY\" \"=_6568a7f155bf754f458c22bcc1da9919\") NIL NIL NIL))\r\n"
@pos: 213
@lex_state: EXPR_DATA
@token.symbol: LPAR
@token.value: "("
```

According to the RFC, Net::IMAP::BodyTypeMessage should only be returned for message/rfc822, otherwise Net::IMAP::BodyTypeBasic:
```
RFC 3501:
  A body type of type MESSAGE and subtype RFC822 contains, immediately after the basic fields,
  the envelope structure, body structure, and size in text lines of the encapsulated message.
  A body type of type TEXT contains, immediately after the basic fields, the size of the body
  in text lines... Extension data follows the basic fields and the type-specific fields listed above
...
    body-type-1part = (body-type-basic / body-type-msg / body-type-text) [SP body-ext-1part]
    body-type-basic = media-basic SP body-fields ; MESSAGE subtype MUST NOT be "RFC822"
    body-type-msg   = media-message SP body-fields SP envelope SP body SP body-fld-lines
    body-type-text  = media-text SP body-fields SP body-fld-lines
    body-ext-1part  = body-fld-md5 [SP body-fld-dsp [SP body-fld-lang [SP body-fld-loc *(SP body-extension)]]] ; MUST NOT be returned on non-extensible "BODY" fetch
    media-basic     = ((DQUOTE ("APPLICATION" / "AUDIO" / "IMAGE" / "MESSAGE" / "VIDEO") DQUOTE) / string) SP media-subtype
    media-message   = DQUOTE "MESSAGE" DQUOTE SP DQUOTE "RFC822" DQUOTE
```


---Files--------------------------------
imap.patch (3.34 KB)


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

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

end of thread, other threads:[~2020-11-24 22:26 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-01 13:41 [ruby-core:100257] [Ruby master Bug#17207] message/disposition-notification: unexpected token LPAR (expected QUOTED or LITERAL) jbecvar
2020-11-24 22:26 ` [ruby-core:101058] " 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).