[sympa-fr] Vérifier les codes de retour des requêtes LDAP

[Sylvain Amrani <adresse@cachée>]

Bonjour,

Il semblerait que les codes de retour des requêtes ldap ne soient pas
examinés dans _include_users_ldap(). Aussi, en cas de limites timelimit
ou sizelimit atteintes, on perdra des abonnés lors du sync_include().

J'obtenais ainsi des nombres d'abonnés flottant pour des grosses listes,
inversement proportionnés à la charge du serveur.

On pourrait ne pas mettre à jour la liste en cas d'erreur :

$ diff -c ../sympa-5.0/src/List.pm List.pm
*** ../sympa-5.0/src/List.pm    2005-04-01 14:33:03.000000000 +0200
--- List.pm     2005-05-26 13:29:12.469386200 +0200
***************
*** 7024,7029 ****
--- 7024,7034 ----
          do_log('debug2',"Unable to perform LDAP search in $ldap_suffix
for $ldap_filter : $@");
          return undef;
      }
+
+     unless ($fetch->code) {
+         do_log('notice','Ldap search failed : %s (searching on server
%s ; suffix %s ; filter %s ; attrs: %s)', $fetch->error(),
join(',',@{$host}), $ldap_suffix, $ldap_filter, $ldap_attrs);
+         return undef;
+     }

      ## Counters.
      my $total = 0;

Je l'ai rapporté sur mantis
http://listes.cru.fr/mantis/view_bug_page.php?f_id=0000418