Project

General

Profile

Development #5425

Ajouter un backend ldap

Added by Jérôme Schneider about 5 years ago. Updated almost 5 years ago.

Status:
Fermé
Priority:
Haut
Assignee:
Jérôme Schneider
Category:
-
Target version:
Start date:
08 Sep 2014
Due date:
% Done:

100%

Patch proposed:
Yes
Planning:
No

Description

Le but de ce backend sera de pouvoir utiliser le mode miroir de OpenLDAP. Ce développement est nécessaire pour qu'on puisse avoir un Mandaye en cluster.

0001-backends-implement-a-new-ldap-storage-backend.patch View (14.3 KB) Jérôme Schneider, 15 Sep 2014 09:33 AM

0001-backends-implement-a-new-ldap-storage-backend.patch View (13.4 KB) Jérôme Schneider, 15 Sep 2014 10:52 AM

Associated revisions

Revision 5294fd40 (diff)
Added by Jérôme Schneider almost 5 years ago

backends: implement a new ldap storage backend

Closes #5425

History

#1 Updated by Jérôme Schneider about 5 years ago

  • Status changed from Nouveau to En cours

#2 Updated by Jérôme Schneider about 5 years ago

  • Target version set to 0.10

#3 Updated by Jérôme Schneider about 5 years ago

#4 Updated by Frédéric Péters about 5 years ago

Sur le schéma c'est étrange d'avoir creationDate et lastConnection nommés ainsi, pourquoi pas createionDate et lastConnectionDate? (et perso j'aurais plutôt mis Time que Date).

Je ne vois pas vraiment l'utilité d'avoir une classe pour n'y mettre que des méthodes statiques mais les choses sont faites ainsi, soit; il y a quelques fautes dans les commentaires, au moins :

  • s/dictionnary/dictionary/
  • s/associations that matching/associations matching/
  • s/conenction/connection/

Et "return a boolean" ce n'est pas vraiment une description pour has_id, ça me semble plutôt être "check the given user is present in the directory".

Finalement ce morceau semble là par erreur, pas de rapport avec le patch :

- mandaye_toolbar_offline = config.getboolean('mandaye', 'offline_toolbar')
+ mandaye_offline_toolbar= config.getboolean('mandaye', 'offline_toolbar')

#5 Updated by Jérôme Schneider about 5 years ago

Un nouveau patch qui :
  • remplace lastConnection par lastConnectionDate (uniquement pour la partie ldap, le reste fera l'objet d'un autre patch)
  • qui corrige les fautes repérées par Fred
  • qui n'inclue plus la correction du settings (mandaye_offline_toolbar) et le replacement d'un print par un logging.

#6 Updated by Frédéric Péters about 5 years ago

ack.

#7 Updated by Jérôme Schneider almost 5 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 60 to 100

Appliqué par commit commit:5294fd40c45b1a5c9f168143aa4dc55f03e81c4e.

#8 Updated by Jérôme Schneider almost 5 years ago

  • Status changed from Résolu (à déployer) to Fermé

Also available in: Atom PDF