[Tickets #837] S/MIME support

bugs at bugs.horde.org bugs at bugs.horde.org
Fri Nov 19 09:01:54 PST 2004


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

Ticket URL: http://bugs.horde.org/ticket/?id=837
-----------------------------------------------------------------------
 Ticket     | 837
 Updated By | ctnpublic-horde at yahoo.com
 Summary    | S/MIME support
 Queue      | IMP
 Version    | HEAD
 State      | Unconfirmed
 Priority   | 2. Medium
 Type       | Bug
 Owners     | 
-----------------------------------------------------------------------


ctnpublic-horde at yahoo.com (2004-11-19 09:01) wrote:

I ran into the same/similar problem.  Diagnosis and suggested fix follow.

There is an assumption in imp/lib/MIME/Viewer/pkcs7.php that the MIME
Content-Type header is in the message body.

Both Microsoft Outlook and IMP generate an S/MIME message with the
Content-Type header in the message header.

This fix picks up the header from the mime_part object, when necessary.

Also add support for (Outlook's) application/pkcs7* content-types.

And added a couple of (missing?) requires.

RCS file: /repository/imp/lib/MIME/Viewer/pkcs7.php,v
retrieving revision 1.49
diff -r1.49 pkcs7.php
108a109
>             case 'application/pkcs7-signature':
112a114
>             case 'application/pkcs7-mime':
251a254
>             require_once IMP_BASE . '/lib/MIME/Headers.php';
253c256,260
<             $headers->addHeader('Content-Type',
$this->_headers['content-type']);
---
>             if (isset($this->_headers['Content-Type'])) {
>                 $headers->addHeader('Content-Type',
$this->_headers['Content-Type']);
>             } else if (isset($this->_headers['content-type'])) {
>                 $headers->addHeader('Content-Type',
$this->_headers['content-type']);
>             }
316c323,326
<                 $this->_headers = &MIME_Structure::parseMIMEHeaders($text,
true, true);
---
>                 require_once 'Horde/MIME/Structure.php';
>                 if (!($this->_headers =
&MIME_Structure::parseMIMEHeaders($text, true, true))) {
>                     $this->_headers = $this->mime_part->header();
>                 }





More information about the bugs mailing list