[Bps-public-commit] r16581 - in RT-Authen-ExternalAuth/trunk: html/Callbacks/ExternalAuth/autohandler

[falcone at bestpractical.com]

Author: falcone
Date: Tue Oct 28 15:03:29 2008
New Revision: 16581

Modified:
   RT-Authen-ExternalAuth/trunk/   (props changed)
   RT-Authen-ExternalAuth/trunk/html/Callbacks/ExternalAuth/autohandler/Auth

Log:
 r41099 at ketch:  falcone | 2008-10-28 15:03:00 -0400
 * add more error handling


Modified: RT-Authen-ExternalAuth/trunk/html/Callbacks/ExternalAuth/autohandler/Auth
==============================================================================
--- RT-Authen-ExternalAuth/trunk/html/Callbacks/ExternalAuth/autohandler/Auth	(original)
+++ RT-Authen-ExternalAuth/trunk/html/Callbacks/ExternalAuth/autohandler/Auth	Tue Oct 28 15:03:29 2008
@@ -61,7 +61,12 @@
 
             # If we autocreated a user, then load the user as the CurrentUser in $session
             # To RT, this means we have a valid, authenticated user
-            $session{'CurrentUser'}->Load($user) if $UserObj->Id;
+            if ($UserObj->Id) {
+                my ($ret, $msg) = $session{'CurrentUser'}->Load($user);
+                unless ($ret) {
+                    $RT::Logger->error("Couldn't load user $user: $msg");
+                }
+            }
         }
     }