Bug #69470
connecteur ldap: module 'ldap' has no attribute 'OPT_X_TLS_REQUIRE_SAN'
Début:
23 septembre 2022
Echéance:
% réalisé:
0%
Temps estimé:
Patch proposed:
Oui
Planning:
Non
Description
Parce qu'on tourne avec python3-ldap version 3.2.0 et que cette option vient avec la 3.4.0
Fichiers
Révisions associées
ldap: enable check hostname only for python-ldap 3.4+ (#69470)
Historique
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-ldap-remove-check-hostname-option-69470.patch 0001-ldap-remove-check-hostname-option-69470.patch ajouté
- Statut changé de Nouveau à Solution proposée
- Patch proposed changé de Non à Oui
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-ldap-remove-check-hostname-option-69470.patch 0001-ldap-remove-check-hostname-option-69470.patch ajouté
L'exception en cas d'inaccessibilité du serveur change un peu aussi avec python-ldap 3.2 ; j’assouplis le test.
Mis à jour par Benjamin Dauvergne il y a plus d'un an
Tu ne voudrais pas juste mettre un check ? C'est quand même utile de valider seulement la clé sans le host, des fois, et la 3.4 arrive dans bookworm.
Avec un help-text adapté :
LDAP_HAS_OPT_X_TLS_REQUIRE_SAN = hasattr(ldap, 'OPT_X_TLS_REQUIRE_SAN')
..
help_text=None if LDAP_HAS_OPT_X_TLS_REQUIRE_SAN else _('python-ldap does not currently handle this option')
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-ldap-enable-check-hostname-only-for-python-ldap-3.4-.patch ajouté
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier
0001-ldap-enable-check-hostname-only-for-python-ldap-3.4-.patchsupprimé
Mis à jour par Thomas Noël il y a plus d'un an
- Fichier 0001-ldap-enable-check-hostname-only-for-python-ldap-3.4-.patch 0001-ldap-enable-check-hostname-only-for-python-ldap-3.4-.patch ajouté
Je prends l'idée, ça fait une migration en moins et ça prépare l'avenir.
Je laisse cependant les tests forcés en python-ldap 3.2 (dans tox.ini) pour qu'on ne se fasse pas avoir à nouveau.
(version blackisée)
Mis à jour par Benjamin Dauvergne il y a plus d'un an
- Statut changé de Solution proposée à Solution validée
Vendu.
Mis à jour par Thomas Noël il y a plus d'un an
- Statut changé de Solution validée à Résolu (à déployer)
commit 66889cb41777b738a06f4d6719d9a056d9e89ae4 (wip/69470-ldap-remove-check-hostname-option)
Author: Thomas NOËL <tnoel@entrouvert.com>
Date: Fri Sep 23 10:03:34 2022 +0200
ldap: enable check hostname only for python-ldap 3.4+ (#69470)
Mis à jour par Transition automatique il y a plus d'un an
- Statut changé de Résolu (à déployer) à Solution déployée
ldap: enable check hostname only for python-ldap 3.4+ (#69470)