Login: blacklisting bei Callback für Redirect zum Ziel implementiert
authorBernd Bleßmann <bernd@kivitendo-premium.de>
Fri, 25 Oct 2019 13:09:46 +0000 (15:09 +0200)
committerBernd Bleßmann <bernd@kivitendo-premium.de>
Fri, 6 Dec 2019 15:26:58 +0000 (16:26 +0100)
SL/Dispatcher.pm

index 16337b2..9aeecfc 100644 (file)
@@ -402,6 +402,7 @@ sub handle_login_error {
   if (   $action =~ m/LoginScreen\/user_login/
       && $params{action}
       && 'get' eq lc($ENV{REQUEST_METHOD})
+      && !_is_callback_blacklisted(map {$_ => $params{$_}} qw(routing_type script controller action) )
   ) {
 
     require SL::Controller::Base;
@@ -419,6 +420,48 @@ sub handle_login_error {
   $self->end_request;
 }
 
+sub _is_callback_blacklisted {
+  my (%params) = @_;
+
+  # You can give a name only, then all actions are blackisted.
+  # Or you can give name and action, then only this action is blacklisted
+  # examples:
+  # {name => 'is',      action => 'edit'}
+  # {name => 'Project', action => 'edit'},
+  my @script_blacklist = (
+    {name => 'admin'},
+    {name => 'login'},
+  );
+
+  my @controller_blacklist = (
+    {name => 'Admin'},
+    {name => 'LoginScreen'},
+  );
+
+  my ($name, $blacklist);
+  if ('old' eq ($params{routing_type} // '')) {
+    $name      = $params{script};
+    $blacklist = \@script_blacklist;
+  } else {
+    $name      = $params{controller};
+    $blacklist = \@controller_blacklist;
+  }
+
+  foreach my $bl (@$blacklist) {
+    return 1 if _is_name_action_blacklisted($bl->{name}, $bl->{action}, $name, $params{action});
+  }
+
+  return;
+}
+
+sub _is_name_action_blacklisted {
+  my ($blacklisted_name, $blacklisted_action, $name, $action) = @_;
+
+  return 1 if ($name // '') eq $blacklisted_name && !$blacklisted_action;
+  return 1 if ($name // '') eq $blacklisted_name && ($action // '') eq $blacklisted_action;
+  return;
+}
+
 sub unrequire_bin_mozilla {
   my $self = shift;
   return unless $self->_interface_is_fcgi;