[Bps-public-commit] r16580 - in RT-Authen-ExternalAuth/trunk: .

[falcone at bestpractical.com]

Author: falcone
Date: Tue Oct 28 15:03:26 2008
New Revision: 16580

Modified:
   RT-Authen-ExternalAuth/trunk/   (props changed)
   RT-Authen-ExternalAuth/trunk/lib/RT/User_Vendor.pm

Log:
 r41098 at ketch:  falcone | 2008-10-28 14:59:37 -0400
 * whine in the logs when a user doesn't specify d_filter


Modified: RT-Authen-ExternalAuth/trunk/lib/RT/User_Vendor.pm
==============================================================================
--- RT-Authen-ExternalAuth/trunk/lib/RT/User_Vendor.pm	(original)
+++ RT-Authen-ExternalAuth/trunk/lib/RT/User_Vendor.pm	Tue Oct 28 15:03:26 2008
@@ -819,6 +819,9 @@
             if ($filter eq "()") { undef($filter) };
             if ($disable_filter eq "()") { undef($disable_filter) };
 
+            unless ($disable_filter) {
+                $RT::Logger->error("You haven't specified a d_filter in your configuration.  Not specifying a d_filter usually results in all users being marked as disabled and being unable to log in");
+            }
 
             if (defined($config->{'attr_map'}->{'Name'})) {
                 # Construct the complex filter