Skip Menu |

This queue is for tickets about the Mail-Box CPAN distribution.

Report information
The Basics
Id: 69516
Status: resolved
Priority: 0/
Queue: Mail-Box

People
Owner: Nobody in particular
Requestors: yyang [...] proofpoint.com
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: (no value)
Fixed in: (no value)



Subject: Mail::Message::Body lost critical information from multipart/signed
If you try to creat a new message body based on a body with the following "Content-Type": Content-Type: multipart/signed; protocol="application/x-pkcs7-signature"; micalg=SHA1; boundary="----=_NextPart_000_008E_01CBD1E6.31A34CE0" the new message body will end up with: Content-Type: multipart/signed; boundary="----=_NextPart_000_008E_01CBD1E6.31A34CE0" The protocol and micalg fields are lost. The following patch fixes the issue: --- lib/Mail/Message/Body/Multipart.pm~ 2011-07-15 11:56:00.000000000 - 0700 +++ lib/Mail/Message/Body/Multipart.pm 2011-07-15 11:55:54.000000000 - 0700 @@ -22,7 +22,7 @@ { my ($self, $args) = @_; my $based = $args->{based_on}; $args->{mime_type} ||= - defined $based ? $based->mimeType : 'multipart/mixed'; + defined $based ? $based->type : 'multipart/mixed'; $self->SUPER::init($args);
Subject: Re: [rt.cpan.org #69516] Mail::Message::Body lost critical information from multipart/signed
Date: Sun, 24 Jul 2011 21:10:39 +0200
To: Yanyan Yang via RT <bug-Mail-Box [...] rt.cpan.org>
From: Mark Overmeer <mark [...] overmeer.net>
* Yanyan Yang via RT (bug-Mail-Box@rt.cpan.org) [110715 21:46]: Show quoted text
> Fri Jul 15 17:46:09 2011: Request 69516 was acted upon. > Transaction: Ticket created by yyang > Queue: Mail-Box > Subject: Mail::Message::Body lost critical information from multipart/signed > Ticket <URL: https://rt.cpan.org/Ticket/Display.html?id=69516 >
Show quoted text
> - defined $based ? $based->mimeType : 'multipart/mixed'; > + defined $based ? $based->type : 'multipart/mixed';
Smart fix! Thanks! Probably needs a regression test :( -- Regards, MarkOv ------------------------------------------------------------------------ Mark Overmeer MSc MARKOV Solutions Mark@Overmeer.net solutions@overmeer.net http://Mark.Overmeer.net http://solutions.overmeer.net
Fixed in 2.101, released Aug 25