X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FMbox.pm;h=84cc384529f174cd020bbfb2360519720b5d0929;hb=3d41aa23f35501ca92aab8aa42980fa73f7fa74f;hp=dc41548b87ce68726dc41aa79bd571589a150641;hpb=d9732203aa53d1fd7b3ba5fae415d3ac003937a0;p=public-inbox.git

diff --git a/lib/PublicInbox/Mbox.pm b/lib/PublicInbox/Mbox.pm
index dc41548b..84cc3845 100644
--- a/lib/PublicInbox/Mbox.pm
+++ b/lib/PublicInbox/Mbox.pm
@@ -1,20 +1,49 @@
-# Copyright (C) 2015 all contributors <meta@public-inbox.org>
-# License: AGPLv3 or later (https://www.gnu.org/licenses/agpl-3.0.txt)
+# Copyright (C) 2015-2018 all contributors <meta@public-inbox.org>
+# License: AGPL-3.0+ <https://www.gnu.org/licenses/agpl-3.0.txt>
 
 # Streaming interface for formatting messages as an mboxrd.
 # Used by the web interface
 package PublicInbox::Mbox;
 use strict;
 use warnings;
-use PublicInbox::MID qw/mid_clean/;
-use URI::Escape qw/uri_escape_utf8/;
-require Email::Simple;
+use PublicInbox::MID qw/mid_clean mid_escape/;
+use PublicInbox::Hval qw/to_filename/;
+use Email::Simple;
+use Email::MIME::Encode;
+
+sub subject_fn ($) {
+	my ($simple) = @_;
+	my $fn = $simple->header('Subject');
+	return 'no-subject' unless defined($fn);
+
+	# no need for full Email::MIME, here
+	if ($fn =~ /=\?/) {
+		eval { $fn = Encode::decode('MIME-Header', $fn) };
+		$fn = 'no-subject' if $@;
+	}
+	$fn =~ s/^re:\s+//i;
+	$fn = to_filename($fn);
+	$fn eq '' ? 'no-subject' : $fn;
+}
 
 sub emit1 {
 	my ($ctx, $msg) = @_;
 	$msg = Email::Simple->new($msg);
-	# single message should be easily renderable in browsers
-	[200, ['Content-Type', 'text/plain'], [ msg_str($ctx, $msg)] ]
+	my $fn = subject_fn($msg);
+	my @hdr = ('Content-Type');
+	if ($ctx->{-inbox}->{obfuscate}) {
+		# obfuscation is stupid, but maybe scrapers are, too...
+		push @hdr, 'application/mbox';
+		$fn .= '.mbox';
+	} else {
+		push @hdr, 'text/plain';
+		$fn .= '.txt';
+	}
+	push @hdr, 'Content-Disposition', "inline; filename=$fn";
+
+	# single message should be easily renderable in browsers,
+	# unless obfuscation is enabled :<
+	[ 200, \@hdr, [ msg_str($ctx, $msg) ] ]
 }
 
 sub msg_str {
@@ -29,7 +58,7 @@ sub msg_str {
 	my $ibx = $ctx->{-inbox};
 	my $base = $ibx->base_url($ctx->{env});
 	my $mid = mid_clean($header_obj->header('Message-ID'));
-	$mid = uri_escape_utf8($mid);
+	$mid = mid_escape($mid);
 	my @append = (
 		'Archived-At', "<$base$mid/>",
 		'List-Archive', "<$base>",
@@ -66,27 +95,28 @@ sub thread_mbox {
 	return sub { need_gzip(@_) } if $@;
 
 	my $cb = sub { $srch->get_thread($ctx->{mid}, @_) };
-	# http://www.iana.org/assignments/media-types/application/gzip
-	[200, ['Content-Type' => 'application/gzip'],
-		PublicInbox::MboxGz->new($ctx, $cb) ];
+	PublicInbox::MboxGz->response($ctx, $cb);
 }
 
 sub emit_range {
 	my ($ctx, $range) = @_;
 
-	eval { require IO::Compress::Gzip };
-	return sub { need_gzip(@_) } if $@;
 	my $query;
 	if ($range eq 'all') { # TODO: YYYY[-MM]
 		$query = '';
 	} else {
 		return [404, [qw(Content-Type text/plain)], []];
 	}
-	my $cb = sub { $ctx->{srch}->query($query, @_) };
+	mbox_all($ctx, $query);
+}
 
-	# http://www.iana.org/assignments/media-types/application/gzip
-	[200, [qw(Content-Type application/gzip)],
-		PublicInbox::MboxGz->new($ctx, $cb) ];
+sub mbox_all {
+	my ($ctx, $query) = @_;
+
+	eval { require IO::Compress::Gzip };
+	return sub { need_gzip(@_) } if $@;
+	my $cb = sub { $ctx->{srch}->query($query, @_) };
+	PublicInbox::MboxGz->response($ctx, $cb, 'results-'.$query);
 }
 
 sub need_gzip {
@@ -106,6 +136,7 @@ EOF
 package PublicInbox::MboxGz;
 use strict;
 use warnings;
+use PublicInbox::Hval qw/to_filename/;
 
 sub new {
 	my ($class, $ctx, $cb) = @_;
@@ -116,10 +147,27 @@ sub new {
 		cb => $cb,
 		ctx => $ctx,
 		msgs => [],
-		opts => { asc => 1, offset => 0 },
+		opts => { offset => 0 },
 	}, $class;
 }
 
+sub response {
+	my ($class, $ctx, $cb, $fn) = @_;
+	my $body = $class->new($ctx, $cb);
+	# http://www.iana.org/assignments/media-types/application/gzip
+	$body->{hdr} = [ 'Content-Type', 'application/gzip' ];
+	$body->{fn} = $fn;
+	my $hdr = $body->getline; # fill in Content-Disposition filename
+	[ 200, $hdr, $body ];
+}
+
+sub set_filename ($$) {
+	my ($fn, $msg) = @_;
+	return to_filename($fn) if defined($fn);
+
+	PublicInbox::Mbox::subject_fn($msg);
+}
+
 # called by Plack::Util::foreach or similar
 sub getline {
 	my ($self) = @_;
@@ -128,10 +176,19 @@ sub getline {
 	my $ibx = $ctx->{-inbox};
 	my $gz = $self->{gz};
 	do {
+		# work on existing result set
 		while (defined(my $smsg = shift @{$self->{msgs}})) {
 			my $msg = eval { $ibx->msg_by_smsg($smsg) } or next;
 			$msg = Email::Simple->new($msg);
 			$gz->write(PublicInbox::Mbox::msg_str($ctx, $msg));
+
+			# use subject of first message as subject
+			if (my $hdr = delete $self->{hdr}) {
+				my $fn = set_filename($self->{fn}, $msg);
+				push @$hdr, 'Content-Disposition',
+						"inline; filename=$fn.mbox.gz";
+				return $hdr;
+			}
 			my $bref = $self->{buf};
 			if (length($$bref) >= 8192) {
 				my $ret = $$bref; # copy :<
@@ -142,6 +199,8 @@ sub getline {
 			# be fair to other clients on public-inbox-httpd:
 			return '';
 		}
+
+		# refill result set
 		$res = $self->{cb}->($self->{opts});
 		$self->{msgs} = $res->{msgs};
 		$res = scalar @{$self->{msgs}};