[rt-users] RT 3.8.2 RC1 and RT-Authen-ExternalAuth 0.07_01

[Emmanuel Lacour]

On Wed, Dec 17, 2008 at 01:42:23PM +0000, Mike Peachey wrote:
> 
> 
> Pretty useless I'm afraid. No idea what would be different between 3.8.1
> and 3.8.2 that would break the password check.. assuming the password it
> correct.
> 

Mike, I just ran into the same bug. The problem is with the code on
top of autohandler/Auth that try to reload RT::User. It reloads it
without User_Vendor.pm from this plugin and so the old IsPassword is
called. I commented this code and my ldap auth is working again.

I run a version of RT pre 3.8.2rc1 (checkout on 20081024).

--- /opt/rt/src/RT-Authen-ExternalAuth-0.07_01/html/Callbacks/ExternalAuth/autohandler/Auth 2008-11-06 21:40:34.000000000 +0100
+++ plugins/RT-Authen-ExternalAuth/html/Callbacks/ExternalAuth/autohandler/Auth 2008-12-17 16:58:34.000000000 +0100
@@ -8,13 +8,13 @@
 # Work around a bug in the RT 3.8.0 and 3.8.1 plugin system (fixed in 3.8.2)
 # Temporarily force RT to reload RT::User, since it isn't being loaded
 # correctly as a plugin.
-unless ($loaded_user) {
-    $RT::Logger->error("Working around bug in RT and reloading RT::User");
-    $loaded_user++;
-    delete $INC{'RT/User.pm'};
-    delete $INC{'RT/User_Overlay.pm'};
-    require RT::User;
-}
+#unless ($loaded_user) {
+#    $RT::Logger->error("Working around bug in RT and reloading RT::User");
+#    $loaded_user++;
+#    delete $INC{'RT/User.pm'};
+#    delete $INC{'RT/User_Overlay.pm'};
+#    require RT::User;
+#}
 ######################################################################################
 
 # If the user is logging in, let's authenticate; if they can auth but don't load