Opened 4 years ago

Closed 4 years ago

#1485847 closed Bugs (fixed)

Messages with 'Content-Disposition=inline' header set, get added to attachments when forwarding

Reported by: arr2036 Owned by:
Priority: 5 Milestone: 0.3-stable
Component: PHP backend Version: 0.2.1
Severity: normal Keywords: Content-Disposition forward attachment
Cc:

Description

As per the title.

There's no reason for Mime parts with Content-Disposition inline to be attached to forwarded email, yet the code suggests they should be.

Just removing the $part->disposition=='inline' stops mail being attached. Does anyone know why this condition is there?

Attachments (1)

compose.inc.patch (574 bytes) - added by arr2036 4 years ago.

Download all attachments as: .zip

Change History (4)

Changed 4 years ago by arr2036

comment:1 Changed 4 years ago by alec

  • Milestone changed from later to 0.2.2

They should be attached if you forward the message (with inline images) in html format.

comment:2 Changed 4 years ago by arr2036

Ok this isn't HTML.

The full condition is:

if (($part->ctype_primary != 'message'
!$bodyIsHtml) &&
($part->disposition=='attachment'
$part->disposition=='inline' $part->headerscontent-id?
(empty($part->disposition) && $part->filename)))

{

So you've already determined that it's not HTML. You can't have inline images in a text email.

comment:3 Changed 4 years ago by alec

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

Fixed in [12217da8].

Note: See TracTickets for help on using tickets.