On prend en compte la gnration concurrente de dn. Certes, c'est moche.

a vite d'utiliser des locks.

darcs-hash:20060907220034-68412-bc06d68a2abdaaa5e31690f9a67d2c334e303807.gz
This commit is contained in:
glondu 2006-09-08 00:00:34 +02:00
parent 9ad806c9f3
commit 5087b93dbd

View file

@ -16,6 +16,7 @@ import ldap, ldap.modlist, ldap_passwd
import config, annuaires, iptools, chgpass, user_tests, cPickle, config_mail import config, annuaires, iptools, chgpass, user_tests, cPickle, config_mail
from chgpass import chgpass from chgpass import chgpass
from affich_tools import coul, prompt from affich_tools import coul, prompt
from syslog import *
date_format = '%d/%m/%Y %H:%M' date_format = '%d/%m/%Y %H:%M'
hostname = gethostname().split(".")[0] hostname = gethostname().split(".")[0]
@ -1124,6 +1125,9 @@ class BaseClasseCrans(CransLdap):
if not self._init_data: if not self._init_data:
### Nouvel enregistrement ### Nouvel enregistrement
# L'enregistrement peut échouer en cas de choix de dn concurrents
# (il n'y a pas de lock sur les dn)
for i in range(0, 5): # 5 tentatives
# Génération du dn # Génération du dn
res = self.conn.search_s(self.base_dn, 2, self.filtre_idn) res = self.conn.search_s(self.base_dn, 2, self.filtre_idn)
vidn = 1 vidn = 1
@ -1145,9 +1149,17 @@ class BaseClasseCrans(CransLdap):
self.dn = '%s=%s,%s' % (self.idn, vidn, self.dn) self.dn = '%s=%s,%s' % (self.idn, vidn, self.dn)
self._data[self.idn]= [ '%d' % vidn ] self._data[self.idn]= [ '%d' % vidn ]
try:
# Ecriture # Ecriture
modlist = ldap.modlist.addModlist(self._data) modlist = ldap.modlist.addModlist(self._data)
self.conn.add_s(self.dn, modlist) self.conn.add_s(self.dn, modlist)
break
except e:
# On logge
syslog("ldap_crans: exception: %s" % e)
if i == 4:
raise
else: else:
### Modification entrée ### Modification entrée
if not self._modifiable: if not self._modifiable: