[Tickets #9099] HTML multipart/alternative should be a pref or mime_driver conf

[bugs at horde.org]

DO NOT REPLY TO THIS MESSAGE. THIS EMAIL ADDRESS IS NOT MONITORED.

Ticket URL: http://bugs.horde.org/ticket/9099
------------------------------------------------------------------------------
  Ticket             | 9099
  Created By         | leandro.damascena at gmail.com
  Summary            | HTML multipart/alternative should be a pref or
                     | mime_driver conf
  Queue              | IMP
  Version            | Git master
  Type               | Enhancement
  State              | New
  Priority           | 1. Low
  Milestone          |
  Patch              |
  Owners             |
------------------------------------------------------------------------------


leandro.damascena at gmail.com (2010-06-16 20:15) wrote:

This is a delicated issue, but I think this is very important for  
IMP.. I will try to explain using my terrible english (lol)..

The IMP uses by default multipart/alternative for images and this  
creates a "trouble" in networks where the server can't access internet  
(outgoing port 80) ... Now I'm working at a company where we have this  
problem, the webserver can't access internet...

Analyzing the code of IMP and trying to find a solution I've  
identified the following item:

"./imp/lib/Compose.php" Line 1081
if (empty($options['nofinal'])) {
	/* Any image links will be downloaded and appended to the
          * message body. */
          $textpart->addPart($this->_convertToMultipartRelated($htmlBody));
} else {
          $textpart->addPart($htmlBody);
}


As my server can't access internet and don't have a proxy server, IMP  
always returns "connect timed out!" when try to get images and doesn't  
send the message.

I propose that this be done by conf file, for example:

if ($conf['compose']['usemultipart']) {
	/* Any image links will be downloaded and appended to the
          * message body. */
         $textpart->addPart($this->_convertToMultipartRelated($htmlBody));
} else {
	$textpart->addPart($htmlBody);
}

Sorry if this already exists in IMP, I couldn't find and I'm sure that  
it is a good discussion.