From 05a99e84eb795dcf19cda5133a4a89a1976fedfc Mon Sep 17 00:00:00 2001 From: "C.J. Adams-Collier" Date: Thu, 18 Sep 2014 11:35:38 -0700 Subject: [PATCH] including error message, and moving session_id reset outside of the conditional block Conflicts: fs_selfservice/FS-SelfService/cgi/selfservice.cgi --- fs_selfservice/FS-SelfService/cgi/selfservice.cgi | 10 +++++++++- 1 file changed, 9 insertions(+), 1 deletion(-) diff --git a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi index 1ec5c0bea..4e21ad8df 100755 --- a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi +++ b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi @@ -124,7 +124,15 @@ unless ( $nologin_actions{$action} ) { 'email' => $email, 'password' => $password ); - $session_id = $login_rv->{'session_id'}; + + if ( $login_rv->{'error'} ) { + my $ip = $cgi->remote_addr(); + warn("login failure [email $email] [ip $ip] [error $login_rv->{error}]"); + } else { + #successful login + } + + $session_id = $login_rv->{'session_id'}; } else { -- 2.11.0