gitea
/
Koha-clms
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Revert "Bug 14408: Allow integers in template paths" 

This reverts commit 253b6f1f51.
This commit is contained in:
Fridolin Somers 2015-06-23 17:49:27 +02:00
parent 1f7fa4fadc
commit a1bc481b33
2 changed files with 2 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
C4/Auth.pm
View File

@ -139,7 +139,7 @@ sub get_template_and_user {
my $in = shift;
my ( $user, $cookie, $sessionID, $flags );
my $safe_chars = 'a-zA-Z0-9_\-\/';
my $safe_chars = 'a-zA-Z_\-\/';
die "bad template path" unless $in->{'template_name'} =~ m/^[$safe_chars]+.tt?$/ig; #sanitize input
$in->{'authnotrequired'} ||= 0;

13
t/db_dependent/Auth.t
View File

@ -8,7 +8,7 @@ use Modern::Perl;
use CGI;
use Test::MockModule;
use List::MoreUtils qw/all any none/;
use Test::More tests => 10;
use Test::More tests => 9;
use Test::Warn;
use C4::Members;
use Koha::AuthUtils qw/hash_password/;
@ -127,17 +127,6 @@ $dbh->{RaiseError} = 1;
};
like ( $@, qr(^bad template path), 'The file $template_name should not be accessible' );
}
( $template, $loggedinuser, $cookies ) = get_template_and_user(
{
template_name => 'errors/500.tt',
query => $query,
type => "intranet",
authnotrequired => 1,
flagsrequired => { catalogue => 1 },
}
);
my $file_exists = ( -f $template->{filename} ) ? 1 : 0;
is ( $file_exists, 1, 'The file errors/500.tt should be accessible (contains integers)' );
}
my $hash1 = hash_password('password');