From 75c383323ae45dd43464281fab45cc7b2950e876 Mon Sep 17 00:00:00 2001 From: Marcel de Rooy Date: Fri, 3 Jun 2016 11:31:44 +0200 Subject: [PATCH] Bug 16644: Plack: Use to_app to remove warning about Plack::App::CGIBin instance Content-Type: text/plain; charset=utf-8 Using Plack with the debian psgi file, I get lots of warnings like: WARNING: Automatically converting Plack::App::CGIBin instance to a PSGI code reference. If you see this warning for each request, you probably need to explicitly call to_app() i.e. Plack::App::CGIBin->new(...)->to_app in your PSGI file. This patch is aimed to eliminate the warns. Test plan: Run Plack with plack.psgi or koha.psgi and verify if you do not see these warnings anymore. --- debian/templates/plack.psgi | 6 +++--- misc/plack/koha.psgi | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/debian/templates/plack.psgi b/debian/templates/plack.psgi index 995fa72..4fb7bf7 100644 --- a/debian/templates/plack.psgi +++ b/debian/templates/plack.psgi @@ -51,15 +51,15 @@ use CGI qw(-utf8 ); # we will loose -utf8 under plack, otherwise my $intranet = Plack::App::CGIBin->new( root => '/usr/share/koha/intranet/cgi-bin' -); +)->to_app; my $opac = Plack::App::CGIBin->new( root => '/usr/share/koha/opac/cgi-bin/opac' -); +)->to_app; # my $api = Plack::App::CGIBin->new( # root => '/usr/share/koha/api/' -# ); +# )->to_app; builder { diff --git a/misc/plack/koha.psgi b/misc/plack/koha.psgi index 7f3ea27..5ffc7fb 100644 --- a/misc/plack/koha.psgi +++ b/misc/plack/koha.psgi @@ -65,7 +65,7 @@ sub watch_for_size { my $CGI_ROOT = $ENV{INTRANET} ? $ENV{INTRANETDIR} : $ENV{OPACDIR}; warn "# using Koha ", $ENV{INTRANET} ? 'intranet' : 'OPAC', " CGI from $CGI_ROOT\n"; -my $app=Plack::App::CGIBin->new(root => $CGI_ROOT); +my $app=Plack::App::CGIBin->new(root => $CGI_ROOT)->to_app; my $home = sub { return [ 302, [ Location => '/cgi-bin/koha/' . ( $ENV{INTRANET} ? 'mainpage.pl' : 'opac-main.pl' ) ] ]; }; -- 1.7.10.4