about summary refs log tree commit homepage
path: root/lib/PublicInbox
diff options
context:
space:
mode:
authorEric Wong <e@80x24.org>2016-05-19 22:02:56 +0000
committerEric Wong <e@80x24.org>2016-05-19 22:02:56 +0000
commitd6cdb106f27abed5d05da035c95e106939fbe3b2 (patch)
tree512df7bdbcd4b4f91af64a4ef9d9d645867a072b /lib/PublicInbox
parent9c54a95e5279826fb8a768f7d6b45cc475415f84 (diff)
downloadpublic-inbox-d6cdb106f27abed5d05da035c95e106939fbe3b2.tar.gz
Having a file start with '.' or '-' can be confusing
and for users, so do not allow it.
Diffstat (limited to 'lib/PublicInbox')
-rw-r--r--lib/PublicInbox/View.pm2
-rw-r--r--lib/PublicInbox/WWW.pm2
2 files changed, 2 insertions, 2 deletions
diff --git a/lib/PublicInbox/View.pm b/lib/PublicInbox/View.pm
index ec5f3907..21949812 100644
--- a/lib/PublicInbox/View.pm
+++ b/lib/PublicInbox/View.pm
@@ -261,7 +261,7 @@ sub attach_link ($$$$) {
         $desc = $fn unless defined $desc;
         $desc = '' unless defined $desc;
         my $sfn;
-        if (defined $fn && $fn =~ /\A[\w\.-]+[a-zA-Z0-9]\z/) {
+        if (defined $fn && $fn =~ /\A[[:alnum:]][\w\.-]+[[:alnum:]]\z/) {
                 $sfn = $fn;
         } elsif ($ct eq 'text/plain') {
                 $sfn = 'a.txt';
diff --git a/lib/PublicInbox/WWW.pm b/lib/PublicInbox/WWW.pm
index 10c2e7ca..5b4d6c18 100644
--- a/lib/PublicInbox/WWW.pm
+++ b/lib/PublicInbox/WWW.pm
@@ -23,7 +23,7 @@ use PublicInbox::GitHTTPBackend;
 our $INBOX_RE = qr!\A/([\w\.\-]+)!;
 our $MID_RE = qr!([^/]+)!;
 our $END_RE = qr!(T/|t/|R/|t\.mbox(?:\.gz)?|t\.atom|raw|)!;
-our $ATTACH_RE = qr!(\d[\.\d]*)-([\w\.-]+[a-zA-Z0-9])!i;
+our $ATTACH_RE = qr!(\d[\.\d]*)-([[:alnum:]][\w\.-]+[[:alnum:]])!i;
 
 sub new {
         my ($class, $pi_config) = @_;