[Bps-public-commit] RT-Extension-LDAPImport branch, master, updated. 0.32_02-2-g56d0d41

Thomas Sibley trs at bestpractical.com
Mon Mar 26 12:08:59 EDT 2012


The branch, master has been updated
       via  56d0d41615e6b23c3201d435f84eeee9819949a7 (commit)
      from  c646bd47a548590a6e3b45caf92494027ca7e7f3 (commit)

Summary of changes:
 bin/rtldapimport.in |    6 ++++++
 1 files changed, 6 insertions(+), 0 deletions(-)

- Log -----------------------------------------------------------------
commit 56d0d41615e6b23c3201d435f84eeee9819949a7
Author: Thomas Sibley <trs at bestpractical.com>
Date:   Mon Mar 26 12:07:15 2012 -0400

    Disable RT::Authen::ExternalAuth while importing users from LDAP
    
    Avoids extra user canonicalization calls which often go back to an
    external info source, such as the same LDAP directory we just fetched
    the info from.
    
    This does mean you can no longer implement configurations where you
    import from LDAP and use ExternalAuth to fetch different user info from
    another source, but I think that's an acceptable tradeoff.

diff --git a/bin/rtldapimport.in b/bin/rtldapimport.in
index 49afa75..efc6c58 100755
--- a/bin/rtldapimport.in
+++ b/bin/rtldapimport.in
@@ -11,6 +11,12 @@ BEGIN {
 use lib '/opt/rt3/local/lib /opt/rt3/lib';
 use RT;
 RT::LoadConfig;
+
+# Avoid extra user canonicalization calls by disabling RT::Authen::ExternalAuth
+# while we import users from LDAP
+ at RT::Plugins = grep { $_ ne 'RT::Authen::ExternalAuth' }
+                    @RT::Plugins;
+
 RT::Init;
 }
 

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



More information about the Bps-public-commit mailing list