Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#1486246 closed Bugs (fixed)

Display message attachment's

Reported by: jasb Owned by:
Priority: 5 Milestone: 0.4-beta
Component: MIME parsing Version: git-master
Severity: normal Keywords:
Cc: keith@…

Description

Howdy,
A message with an attach, instead of showing the message and the attach, RC show's the message, and the attach below the message.
I've tried one of the three configuration options and none worked.
--
0 - Full RFC 2231 compatible
1 - RFC 2047 for 'name' and RFC 2231 for 'filename' parameter (Thunderbird's
default)
2 - Full 2047 compatible
--

going to attach a raw message, that show's the attach message on any email client, MSO2007, SquirrelMail?, Windows Mail (former Outlook Express), ThunderBird?.

Attachments (13)

raw msg, not displaing attachs.txt (48.0 KB) - added by jasb 4 years ago.
sc1.JPG (68.1 KB) - added by jasb 4 years ago.
sc2.JPG (127.1 KB) - added by jasb 4 years ago.
sc3.JPG (197.4 KB) - added by jasb 4 years ago.
rcube_message.patch (1.5 KB) - added by alec 4 years ago.
sc1_2.JPG (78.9 KB) - added by jasb 4 years ago.
sc2_2.JPG (151.4 KB) - added by jasb 4 years ago.
sc3_2.JPG (199.2 KB) - added by jasb 4 years ago.
mso.JPG (72.1 KB) - added by jasb 4 years ago.
Assp-user Digest, Vol 44, Issue 4.eml (18.8 KB) - added by peridoc 4 years ago.
msg_r3154_outlook.JPG (108.1 KB) - added by jasb 4 years ago.
msg_r3154_raw_msg.txt (10.9 KB) - added by jasb 4 years ago.
msg_r3154_roundcube.JPG (151.9 KB) - added by jasb 4 years ago.

Download all attachments as: .zip

Change History (27)

Changed 4 years ago by jasb

comment:1 Changed 4 years ago by alec

  • Component changed from User Interface to MIME parsing
  • Milestone changed from later to 0.3.1

Attach BODYSTRUCTURE and some screenshots. It looks ok here. The options that you're talking about are not for messages displaying.

Changed 4 years ago by jasb

Changed 4 years ago by jasb

Changed 4 years ago by jasb

comment:2 Changed 4 years ago by jasb

Hi Allec,
See the images I attached in the order (sc1/sc2/sc3).

comment:3 Changed 4 years ago by alec

So, this looks almost like in Thunderbird2, but what we have here... A message with two parts. First part is displayed correctly. Second part is of type message/rfc822. Roundcube and Thunderbird are displaying such parts even with disposition=attachment set. Now, this part contains three subparts. First is displayed correctly. Second is a "Delivery report.dat" attachment, this is also ok, but the 3rd subpart is also message/rfc822. Here Roundcube is displaying both parts, text/plain and text/html and this is a bug. A second difference with TB is that TB lists disposition=attachment parts on attachment list, but should we do this if we're displaying their content?

Changed 4 years ago by alec

comment:4 Changed 4 years ago by alec

Attached simple patch, but here I'm not sure if this not breaks something else.

Changed 4 years ago by jasb

Changed 4 years ago by jasb

Changed 4 years ago by jasb

comment:5 Changed 4 years ago by jasb

Check the new screenshot's.
I've applied your patch, something is different now.
Is there a way to add an option to show the 1st attach like MSOutlook does?
Show the attach only, and if I want to open the attach it'll let me?
Going to attach a screenshot from MSO.

Changed 4 years ago by jasb

comment:6 Changed 4 years ago by alec

(My patch applied in [3815e050]). So, you just want to display all message/rfc822 parts as attachments on attachments list instead displaying them in the message window, below the main message?

comment:7 Changed 4 years ago by peridoc

  • Cc keith@… added

I saw that this updated in the trunk last night and as of today I am not able to read my Sourceforge.net mail list digest emails that are sent in MIME format. I was able to read them yesterday before updating the code from the trunk (updated from the previous day's trunk version). Could this change have possibly broken reading them? Do I need to supply any items to help debug? Thnaks!

comment:8 follow-up: Changed 4 years ago by alec

@peridoc: yes, attache sample message, please.

Changed 4 years ago by peridoc

comment:9 Changed 4 years ago by jasb

Hi Allec,
Something's different now, but i believe it still need's some tune.
Check my screenshot's, and raw msg.

Changed 4 years ago by jasb

Changed 4 years ago by jasb

Changed 4 years ago by jasb

comment:10 in reply to: ↑ 8 Changed 4 years ago by peridoc

Replying to alec:

@peridoc: yes, attache sample message, please.

I forgot to mention that I attached the sample message as requested. Sorry.

Thanks!

comment:11 Changed 4 years ago by alec

Fixed in [19d6a893]. @jasb: what you expect?

comment:12 Changed 4 years ago by jasb

Alec,
It's perfect now!
Thank you!

comment:13 Changed 4 years ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

comment:14 Changed 4 years ago by peridoc

I am fixed now too. Thank you very much!

Note: See TracTickets for help on using tickets.