Bug 17776: (QA follow-up) Consistent regex for Plack detection
authorMarcel de Rooy <m.de.rooy@rijksmuseum.nl>
Fri, 5 Oct 2018 08:25:43 +0000 (10:25 +0200)
committerNick Clemens <nick@bywatersolutions.com>
Tue, 9 Oct 2018 15:02:49 +0000 (15:02 +0000)
Synchronizing:
C4/Auth_with_shibboleth.pm:    if ( any { /(^psgi|^plack)/i } keys %ENV ) {
Koha/AuthUtils.pm:    if ( ( any { /(^psgi\.|^plack\.)/i } keys %ENV ) && $ENV{SCRIPT_NAME} =~ m,^/(intranet|opac)(.*), ) {
about.pl:if ( any { /(^psgi\.|^plack\.)/i } keys %ENV ) {

Actually we should move it to a subroutine. New report please.

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com>

Signed-off-by: Nick Clemens <nick@bywatersolutions.com>

C4/Auth_with_shibboleth.pm

index dc18a1f..480d5d8 100644 (file)
@@ -84,7 +84,7 @@ sub get_login_shib {
 
     my $matchAttribute = $config->{mapping}->{ $config->{matchpoint} }->{is};
 
-    if ( any { /(^psgi|^plack)/i } keys %ENV ) {
+    if ( any { /(^psgi\.|^plack\.)/i } keys %ENV ) {
       $debug and warn $matchAttribute . " value: " . $ENV{"HTTP_".uc($matchAttribute)};
       return $ENV{"HTTP_".uc($matchAttribute)} || '';
     } else {