Ticket #1485468 (closed Patches: fixed)

Opened 3 months ago

Last modified 3 months ago

Broken attach file name

Reported by: joungkyun Owned by:
Priority: 10 - Lowest Milestone: 0.2-stable
Component: MIME parsing Version: 0.2-beta
Severity: major Keywords:
Cc:

Description

On SVN trunk revision 1941 or 0.2-beta, attach file name that made with RFC 2231 is broken.

For example, file name encoding with follows,

 filename*0*=UTF-8''%32%30%30%38%30%39%32%36%20%EC%82%AC%EB%AC%B4%EC%8B%A4;
 filename*1*=%2E%6A%70%67

roundcube parsed only filename*1* with follow codes

     else if (!empty($part->d_parameters['filename*0*'])) {
       $i = 0;
       while (isset($part->d_parameters['filename*'.$i.'*'])) {
         $i++;
         $filename_encoded .= $part->d_parameters['filename*'.$i.'*'];
       }
       if ($i<2) {
         $headers = iil_C_FetchPartHeader($this->conn, $this->mailbox, $this->_msg_id, $part->mime_id);

So, I attach patch that fix this problem.

Attachments

rcube-svn1941-broken-attachname.patch (1.8 kB) - added by joungkyun 3 months ago.
fixed broken attach file name with RFC 2231
rcube_0.2_broken_attachname.jpg (20.0 kB) - added by joungkyun 3 months ago.
rcube_0.2_good_attachname.jpg (19.9 kB) - added by joungkyun 3 months ago.

Change History

Changed 3 months ago by joungkyun

fixed broken attach file name with RFC 2231

Changed 3 months ago by joungkyun

Changed 3 months ago by joungkyun

Changed 3 months ago by joungkyun

filename of All charset has this problem.

Changed 3 months ago by tensor

Please attach the source of sample message.

Changed 3 months ago by alec

  • status changed from new to closed
  • resolution set to fixed
  • component changed from Client Scripts to MIME parsing

Fixed in r1944.

Note: See TracTickets for help on using tickets.