X-Git-Url: http://www.git.stargrave.org/?a=blobdiff_plain;f=lib%2FPublicInbox%2FReply.pm;h=d96fadfc8b3bfc8eec1c8400218a88fa4280f77b;hb=5c8909925072804901e9c3b45bbf25446d379e7b;hp=0cd36fdad542ee53e2cd9d95b3ef60bedc9d74e9;hpb=666844ae42b5b17fd2e9b7ea6cf04727487b5322;p=public-inbox.git
diff --git a/lib/PublicInbox/Reply.pm b/lib/PublicInbox/Reply.pm
index 0cd36fda..d96fadfc 100644
--- a/lib/PublicInbox/Reply.pm
+++ b/lib/PublicInbox/Reply.pm
@@ -1,21 +1,17 @@
-# Copyright (C) 2014-2017 all contributors
+# Copyright (C) 2014-2021 all contributors
# License: AGPL-3.0+
+
+# For reply instructions and address generation in WWW UI
package PublicInbox::Reply;
use strict;
use warnings;
use URI::Escape qw/uri_escape_utf8/;
-use PublicInbox::Hval qw/ascii_html/;
+use PublicInbox::Hval qw(ascii_html obfuscate_addrs mid_href);
use PublicInbox::Address;
-use PublicInbox::MID qw/mid_clean mid_escape/;
+use PublicInbox::MID qw(mid_clean);
+use PublicInbox::Config;
-sub squote_maybe ($) {
- my ($val) = @_;
- if ($val =~ m{([^\w@\./,\%\+\-])}) {
- $val =~ s/(['!])/'\\$1'/g; # '!' for csh
- return "'$val'";
- }
- $val;
-}
+*squote_maybe = \&PublicInbox::Config::squote_maybe;
sub add_addrs {
my ($to, $cc, @addrs) = @_;
@@ -33,8 +29,17 @@ sub mailto_arg_link {
my ($ibx, $hdr) = @_;
my $cc = {}; # everyone else
my $to; # this is the From address by default
+ my $reply_to_all = 'reply-to-all'; # the only good default :P
+ my $reply_to_cfg = $ibx->{replyto};
- foreach my $rt (split(/\s*,\s*/, $ibx->{replyto} || ':all')) {
+ $reply_to_cfg ||= ':all';
+ if ($reply_to_cfg =~ /\A:none=(.*)/) {
+ my $msg = $1;
+ $msg = 'replies disabled' if $msg eq '';
+ return \$msg;
+ }
+
+ foreach my $rt (split(/\s*,\s*/, $reply_to_cfg)) {
if ($rt eq ':all') {
foreach my $h (@reply_headers) {
my $v = $hdr->header($h);
@@ -43,8 +48,10 @@ sub mailto_arg_link {
add_addrs(\$to, $cc, @addrs);
}
} elsif ($rt eq ':list') {
+ $reply_to_all = 'reply-to-list';
add_addrs(\$to, $cc, $ibx->{-primary_address});
} elsif ($rt =~ /\A(?:$reply_headers)\z/io) {
+ # ugh, this is weird...
my $v = $hdr->header($rt);
if (defined($v) && ($v ne '')) {
my @addrs = PublicInbox::Address::emails($v);
@@ -61,14 +68,17 @@ sub mailto_arg_link {
my $obfs = $ibx->{obfuscate};
my $subj = $hdr->header('Subject') || '';
$subj = "Re: $subj" unless $subj =~ /\bRe:/i;
+ my $subj_raw = $subj;
my $mid = $hdr->header_raw('Message-ID');
push @arg, '--in-reply-to='.squote_maybe(mid_clean($mid));
- my $irt = mid_escape($mid);
+ my $irt = mid_href($mid);
+ add_addrs(\$to, $cc, $ibx->{-primary_address}) unless defined($to);
delete $cc->{$to};
if ($obfs) {
my $arg_to = $to;
- $arg_to =~ s/\./\$(echo .)/;
+ obfuscate_addrs($ibx, $arg_to, '$(echo .)');
push @arg, "--to=$arg_to";
+ # no $subj for $href below
} else {
push @arg, "--to=$to";
$to = uri_escape_utf8($to);
@@ -79,8 +89,9 @@ sub mailto_arg_link {
if (@cc) {
if ($obfs) {
push(@arg, map {
- s/\./\$(echo .)/;
- "--cc=$_";
+ my $addr = $_;
+ obfuscate_addrs($ibx, $addr, '$(echo .)');
+ "--cc=$addr";
} @cc);
} else {
$cc = '&Cc=' . uri_escape_utf8(join(',', @cc));
@@ -88,16 +99,18 @@ sub mailto_arg_link {
}
}
+ push @arg, "--subject=".squote_maybe($subj_raw);
+
# I'm not sure if address obfuscation and mailto: links can
# be made compatible; and address obfuscation is misguided,
# anyways.
- return (\@arg, '') if $obfs;
+ return (\@arg, '', $reply_to_all) if $obfs;
# order matters, Subject is the least important header,
# so it is last in case it's lost/truncated in a copy+paste
my $href = "mailto:$to?In-Reply-To=$irt${cc}&Subject=$subj";
- (\@arg, ascii_html($href));
+ (\@arg, ascii_html($href), $reply_to_all);
}
1;