Skip Menu |

Preferred bug tracker

Please visit the preferred bug tracker to report your issue.

This queue is for tickets about the Email-MIME-Attachment-Stripper CPAN distribution.

Report information
The Basics
Id: 61709
Status: open
Priority: 0/
Queue: Email-MIME-Attachment-Stripper

People
Owner: Nobody in particular
Requestors: icamargo [...] st-andes.com
Cc:
AdminCc:

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



Subject: multipart/alternative patch (and maybe others).
Hi! I saw these issues: 1. When including a plain text file, it was *not* detached. 2. The module removed the html version of the message. So, I modified it a little, and it looks like it "works for me" now. Please, take a look at it! it is just 1 line change. Also: let me know if you find something odd with my "quick-n-dirty" fix, please? Sincerely, Ildefonso Camargo
Subject: multipart-alternative.diff
--- /root/.cpan/build/Email-MIME-Attachment-Stripper-1.316-6DA6Ou/lib/Email/MIME/Attachment/Stripper.pm 2009-01-23 22:50:40.000000000 -0430 +++ Stripper.pm 2010-09-27 18:07:14.000000000 -0430 @@ -113,9 +113,9 @@ foreach ( $part->parts ) { my $ct = $_->content_type || 'text/plain'; my $dp = $_->header('Content-Disposition') || 'inline'; - + # JICT --- Changed some things here. push(@keep, $_) and next - if $ct =~ m[text/plain]i && $dp =~ /inline/i; + if $ct =~ $ct =~ m[multipart/alternative]i || $dp =~ /inline/i; push @attach, $_; if ($_->parts > 1) { my @kept=$self->_detach_all($_);
I will review this ticket soon, but in the meantime, E-M-A-S may be replaced or significantly rewritten: https://github.com/rjbs/Email-MIME-Attachment-Stripper/issues/2 -- rjbs