From b8a5194d24cb8ce5c2f7687376cdcd24cd52e67a Mon Sep 17 00:00:00 2001 From: Martin Renvoize Date: Wed, 6 Nov 2019 12:09:49 +0000 Subject: [PATCH] Bug 23068: (RM follow-up) Make Net::Netmask required Net::Netmask is used in the new Plack::Middleware::RealIP module, which is in turn used in the psgi plack startup script. We don't have access to C4::Context inside the startup script and as such don't have access to the config to work out if loading this module should be optional. As such, I'm taking the easy route and just making the dependancy a required one. It's small and already packaged for debian and as such should not cause anyone any issues. Signed-off-by: Martin Renvoize --- C4/Installer/PerlDependencies.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/C4/Installer/PerlDependencies.pm b/C4/Installer/PerlDependencies.pm index a62e633161..d553961cc5 100644 --- a/C4/Installer/PerlDependencies.pm +++ b/C4/Installer/PerlDependencies.pm @@ -910,7 +910,7 @@ our $PERL_DEPS = { }, 'Net::Netmask' => { 'usage' => 'Koha X-Forwarded-For support', - 'required' => '0', + 'required' => '1', 'min_ver' => '1.9022', }, 'Net::Z3950::SimpleServer' => { -- 2.39.2