Project

General

Profile

Tâche #7928

Distribution EOLE - Scénario #16720: Traitement express (28-35)

le contenu de la valeur "ldap_base_dn" n'est pas testé mais peu provoquer des plantages

Added by Emmanuel GARETTE over 7 years ago. Updated about 5 years ago.

Status:
Fermé
Priority:
Normal
Assigned To:
Start date:
07/22/2016
Due date:
% Done:

100%

Estimated time:
2.00 h
Spent time:
Remaining (hours):
0.0

Description

Avec un CreoleLint qui fait plus de test, j'obtiens l'erreur suivante :

/usr/share/eole/creole/distrib/00_annuaire.ldif : ('Il y a une erreur', IndexError('list index out of range',), 'avec le dictionnaire', {'activer_client_ldap': u'local', 'ldap_base_dn': u'value'})

En effet, la fonction get_organization ne prend pas compte les valeurs farfelues :

%def %%get_organization()
%return %%ldap_base_dn.split(',')[0].split('=')[1]
%end def

Le mieux serait de faire une validation sur le contenu de la variable.

Associated revisions

Revision abdec51e (diff)
Added by Joël Cuissinat about 5 years ago

Ajout d'une validation sur le Base DN

Ref: #7928 @30m

Revision b3b06bbb (diff)
Added by Joël Cuissinat about 5 years ago

Validation du Base DN déplacée dans eole-client-annuaire

Ref: #7928 @20m

Revision c70c5229 (diff)
Added by Joël Cuissinat about 5 years ago

"ou" tolérées pour la variable ldap_base_dn

Ref: #7928 @30m

History

#1 Updated by Joël Cuissinat about 5 years ago

  • Tracker changed from Anomalie to Tâche
  • Estimated time set to 2.00 h
  • Parent task set to #16720
  • Remaining (hours) set to 2.0

#2 Updated by Joël Cuissinat about 5 years ago

  • Status changed from Nouveau to En cours
  • Assigned To set to Joël Cuissinat
  • Priority changed from Haut to Normal
  • Start date set to 07/22/2016

#3 Updated by Joël Cuissinat about 5 years ago

Pour reproduire : Eolebase + eole-annuaire :

root@eolebase:~# CreoleSet ldap_base_dn toto 
root@eolebase:~# CreoleCat -t 00_annuaire.ldif
root - Erreur lors de l'instanciation du template de /var/lib/eole/00_annuaire.ldif : list index out of range

#4 Updated by Joël Cuissinat about 5 years ago

  • Status changed from En cours to Résolu
  • % Done changed from 0 to 100
  • Remaining (hours) changed from 2.0 to 0.25

#5 Updated by Joël Cuissinat about 5 years ago

  • Status changed from Résolu to En cours

#6 Updated by Joël Cuissinat about 5 years ago

  • % Done changed from 100 to 50

Visiblement au MEEM, ils renseignent potentiellement ce genre de valeur ?

ou=TEST-PNESR,ou=domaines,ou=Samba,ou=applications,ou=ressources,dc=equipement,dc=gouv,dc=fr

On pourrait exceptionnellement autoriser "ou=" (à vérifier sur eCDL)

Confirmé sur eCDL 2.6.0 :

ValueError: valeur invalide pour l'option ldap_base_dn : La racine doit débuter par une organisation (o=) ou une composante du domaine (dc=)

/usr/share/eole/creole/dicos/30_ecdl.xml:            <variable name='ldap_base_dn' redefine='True'>
/usr/share/eole/creole/dicos/30_ecdl.xml-                <value>ou=domaines,ou=Samba,ou=applications,ou=ressources,dc=equipement,dc=gouv,dc=fr</value>
/usr/share/eole/creole/dicos/30_ecdl.xml-            </variable>

#7 Updated by Joël Cuissinat about 5 years ago

  • Status changed from En cours to Résolu
  • % Done changed from 50 to 100

#8 Updated by Emmanuel GARETTE about 5 years ago

  • Status changed from Résolu to Fermé
  • Remaining (hours) changed from 0.25 to 0.0

OK :

root@eolebase:~# CreoleGet ldap_base_dn
o=gouv,c=fr
root@eolebase:~# CreoleSet ldap_base_dn toto
Valeur invalide : valeur invalide pour l'option ldap_base_dn : La racine doit débuter par une organisation (o=), une composante du domaine (dc=) ou une unité organisationnelle (ou=)
root@eolebase:~# CreoleSet ldap_base_dn ou=toto
root@eolebase:~# CreoleCat -t 00_annuaire.ldif
root@eolebase:~# CreoleSet ldap_base_dn ou=TEST-PNESR,ou=domaines,ou=Samba,ou=applications,ou=ressources,dc=equipement,dc=gouv,dc=fr
root@eolebase:~# CreoleGet ldap_base_dn
ou=TEST-PNESR,ou=domaines,ou=Samba,ou=applications,ou=ressources,dc=equipement,dc=gouv,dc=fr
root@eolebase:~# CreoleCat -t 00_annuaire.ldif
root@eolebase:~# 

Also available in: Atom PDF