From 7ed251ce3ffbc41cf8a09ac4b5dffc292efa0c67 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Cl=C3=A9ment=20OUDOT?= Date: Mon, 4 May 2020 21:48:06 +0200 Subject: [PATCH] Better log messages (#2165) --- .../lib/Lemonldap/NG/Portal/Auth/Combination.pm | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Auth/Combination.pm b/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Auth/Combination.pm index f99c95503..55968a64e 100644 --- a/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Auth/Combination.pm +++ b/lemonldap-ng-portal/lib/Lemonldap/NG/Portal/Auth/Combination.pm @@ -6,7 +6,7 @@ use Lemonldap::NG::Common::Combination::Parser; use Lemonldap::NG::Portal::Main::Constants qw(PE_OK PE_ERROR PE_FIRSTACCESS); use Scalar::Util 'weaken'; -our $VERSION = '2.0.7'; +our $VERSION = '2.0.8'; # TODO: See Lib::Wrapper extends 'Lemonldap::NG::Portal::Main::Auth'; @@ -142,8 +142,9 @@ sub authLogout { ->[0]; unless ($sub) { $self->logger->warn( -"Condition changed between and logout, unable to select good backend" - ); + "Condition changed between login and logout for " + . $req->user + . ", unable to select good backend" ); return PE_ERROR; } my ( $res, $name ) = $sub->( 'authLogout', @_ ); @@ -218,8 +219,9 @@ sub try { # If more than 1 scheme is available my ( $res, $name ); unless ( ref $stack->[$nb]->[$type] ) { - $self->logger->error( -'Something went wrong in combination, unable to find any auth scheme (try == ' + $self->logger->error( 'Something went wrong in combination for ' + . $req->user + . ', unable to find any auth scheme (try == ' . ( $nb + 1 ) . ')' ); return PE_ERROR;