Commit d8961668 authored by Thomas Sibley's avatar Thomas Sibley
Browse files

Lower log level to debug for skipped numeric users/groups

It's not easy to exclude numeric users/groups with an LDAP filter,
leaving no way of acknowledging and suppressing the warning.  Rather
than add an option, just lower the log level.

While debugging it's helpful to see what was skipped, but under normal
usage (i.e. from cron), you want to ensure you see meaningful errors and
warnings not the same ones every night.
parent 87a75f13
......@@ -435,7 +435,7 @@ sub import_users {
next;
}
if ( $user->{Name} =~ /^[0-9]+$/) {
$self->_warn("Skipping user '$user->{Name}', as it is numeric");
$self->_debug("Skipping user '$user->{Name}', as it is numeric");
next;
}
$self->_import_user( user => $user, ldap_entry => $entry, import => $args{import} );
......@@ -934,7 +934,7 @@ sub import_groups {
next;
}
if ( $group->{Name} =~ /^[0-9]+$/) {
$self->_warn("Skipping group '$group->{Name}', as it is numeric");
$self->_debug("Skipping group '$group->{Name}', as it is numeric");
next;
}
$self->_import_group( %args, group => $group, ldap_entry => $entry );
......
Supports Markdown
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment