[freeside-commits] branch FREESIDE_3_BRANCH updated. 05a99e84eb795dcf19cda5133a4a89a1976fedfc

[Carl J. Adams-Collier]

The branch, FREESIDE_3_BRANCH has been updated
       via  05a99e84eb795dcf19cda5133a4a89a1976fedfc (commit)
      from  9931f703daa44a56da2382fd823271eff138959a (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 05a99e84eb795dcf19cda5133a4a89a1976fedfc
Author: C.J. Adams-Collier <cjac at colliertech.org>
Date:   Thu Sep 18 11:35:38 2014 -0700

    including error message, and moving session_id reset outside of the conditional block
    
    Conflicts:
    	fs_selfservice/FS-SelfService/cgi/selfservice.cgi

diff --git a/fs_selfservice/FS-SelfService/cgi/selfservice.cgi b/fs_selfservice/FS-SelfService/cgi/selfservice.cgi
index 1ec5c0b..4e21ad8 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 {
 

-----------------------------------------------------------------------

Summary of changes:
 fs_selfservice/FS-SelfService/cgi/selfservice.cgi |   10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)