From 3094dbe3c102af4cf9780b6f8059978afb267d6d Mon Sep 17 00:00:00 2001 From: Moritz Bunkus Date: Thu, 6 Jun 2019 12:27:14 +0200 Subject: [PATCH] =?utf8?q?Admin-Controller:=20falscher=20Funktionsname=20f?= =?utf8?q?=C3=BCr=20Auth-Variablen=20gefixt?= MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Die Umstellung auf Auth-Handler mit Einführung der Möglichkeit, sich auch über HTTP-Basic-Authentifizierung anzumelden, hat als Änderung mitgebracht, dass die Authentfizieriungsvariablen (z.B. »{AUTH}admin_password«) nicht mehr in $::form verbleiben. Einige Controller benötigen diese aber; z.B. der Admin-Controller, wenn es noch keine Auth-DB oder keine Session-Tabellen gibt — denn dann ist die einzige Möglichkeit, das von der Benutzer*in eingegebene Admin-Passwort über mehrere Requests zu erhalten, es in der Form mitzuschleifen. Der Admin-Controller war darauf auch schon vorbereitet — leider hatte aber die Funktion, die diesen Umstand mitteilt, den falschen Namen und wurde somit niemals aufgerufen. Dies ist der zweite Teil des Fixes von #376. --- SL/Controller/Admin.pm | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/SL/Controller/Admin.pm b/SL/Controller/Admin.pm index 0fd15660d..f3e634ff1 100644 --- a/SL/Controller/Admin.pm +++ b/SL/Controller/Admin.pm @@ -31,7 +31,7 @@ __PACKAGE__->run_before(\&setup_layout); __PACKAGE__->run_before(\&setup_client, only => [ qw(list_printers new_printer edit_printer save_printer delete_printer) ]); sub get_auth_level { "admin" }; -sub keep_auth_vars { +sub keep_auth_vars_in_form { my ($class, %params) = @_; return $params{action} eq 'login'; } -- 2.20.1