[Rt-commit] rt branch 5.0/remove-ldapimport-numeric-username-limit created. rt-5.0.3-125-g738be6f09d

[BPS Git Server]

This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "rt".

The branch, 5.0/remove-ldapimport-numeric-username-limit has been created
        at  738be6f09de72cb39571dbf4207f8e257ab4063f (commit)

- Log -----------------------------------------------------------------
commit 738be6f09de72cb39571dbf4207f8e257ab4063f
Author: Blaine Motsinger <blaine at bestpractical.com>
Date:   Wed Sep 28 13:08:09 2022 -0500

    Remove LDAPImport numeric username limit
    
    Usernames may be updated to all numeric during ExternalAuth user
    update.  This commit removes the check for numeric username from
    the LDAPImport user update to make it consistent with ExternalAuth.

diff --git a/lib/RT/LDAPImport.pm b/lib/RT/LDAPImport.pm
index b65e6cffe5..1f213e61d7 100644
--- a/lib/RT/LDAPImport.pm
+++ b/lib/RT/LDAPImport.pm
@@ -624,10 +624,6 @@ sub _import_user {
         $RT::Logger->warn("No Name or Emailaddress for user, skipping ".Dumper($args{user}));
         return;
     }
-    if ( $args{user}{Name} =~ /^[0-9]+$/) {
-        $RT::Logger->debug("Skipping user '$args{user}{Name}', as it is numeric");
-        return;
-    }
 
     $RT::Logger->debug("Processing user $args{user}{Name}");
     $self->_cache_user( %args );

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


hooks/post-receive
-- 
rt