]> Sergey Matveev's repositories - public-inbox.git/blobdiff - lib/PublicInbox/WwwAttach.pm
run update-copyrights from gnulib for 2019
[public-inbox.git] / lib / PublicInbox / WwwAttach.pm
index 5cf56a804ea77d21a255c83476e3e9535e9855af..2de568041f31a17b28f76faf28e68a892b755a5e 100644 (file)
@@ -1,24 +1,21 @@
-# Copyright (C) 2016 all contributors <meta@public-inbox.org>
+# Copyright (C) 2016-2019 all contributors <meta@public-inbox.org>
 # License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # For retrieving attachments from messages in the WWW interface
 package PublicInbox::WwwAttach; # internal package
 use strict;
 use warnings;
-use Email::MIME;
+use bytes (); # only for bytes::length
 use Email::MIME::ContentType qw(parse_content_type);
-$Email::MIME::ContentType::STRICT_PARAMS = 0;
-use PublicInbox::MID qw(mid2path);
+use PublicInbox::MIME;
 use PublicInbox::MsgIter;
 
 # /$LISTNAME/$MESSAGE_ID/$IDX-$FILENAME
 sub get_attach ($$$) {
        my ($ctx, $idx, $fn) = @_;
-       my $path = mid2path($ctx->{mid});
-
        my $res = [ 404, [ 'Content-Type', 'text/plain' ], [ "Not found\n" ] ];
-       my $mime = $ctx->{git}->cat_file("HEAD:$path") or return $res;
-       $mime = Email::MIME->new($mime);
+       my $mime = $ctx->{-inbox}->msg_by_mid($ctx->{mid}) or return $res;
+       $mime = PublicInbox::MIME->new($mime);
        msg_iter($mime, sub {
                my ($part, $depth, @idx) = @{$_[0]};
                return if join('.', @idx) ne $idx;
@@ -30,7 +27,7 @@ sub get_attach ($$$) {
                if ($ct && (($ct->{discrete} || '') eq 'text')) {
                        # display all text as text/plain:
                        my $cset = $ct->{attributes}->{charset};
-                       if ($cset && ($cset =~ /\A[\w-]+\z/)) {
+                       if ($cset && ($cset =~ /\A[a-zA-Z0-9_\-]+\z/)) {
                                $res->[1]->[1] .= qq(; charset=$cset);
                        }
                } else { # TODO: allow user to configure safe types