Project

General

Profile

Bug #69470

connecteur ldap: module 'ldap' has no attribute 'OPT_X_TLS_REQUIRE_SAN'

Added by Thomas Noël 6 days ago. Updated 6 days ago.

Status:
Solution déployée
Priority:
Normal
Assignee:
Target version:
-
Start date:
23 September 2022
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

Parce qu'on tourne avec python3-ldap version 3.2.0 et que cette option vient avec la 3.4.0


Files

Associated revisions

Revision 66889cb4 (diff)
Added by Thomas Noël 6 days ago

ldap: enable check hostname only for python-ldap 3.4+ (#69470)

History

#1

Updated by Thomas Noël 6 days ago

#2

Updated by Thomas Noël 6 days ago

  • Assignee set to Thomas Noël
#3

Updated by Thomas Noël 6 days ago

L'exception en cas d'inaccessibilité du serveur change un peu aussi avec python-ldap 3.2 ; j’assouplis le test.

#4

Updated by Benjamin Dauvergne 6 days ago

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')

#5

Updated by Thomas Noël 6 days ago

  • File 0001-ldap-enable-check-hostname-only-for-python-ldap-3.4-.patch added
#6

Updated by Thomas Noël 6 days ago

  • File deleted (0001-ldap-enable-check-hostname-only-for-python-ldap-3.4-.patch)
#7

Updated by Thomas Noël 6 days ago

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)

#8

Updated by Benjamin Dauvergne 6 days ago

  • Status changed from Solution proposée to Solution validée

Vendu.

#9

Updated by Thomas Noël 6 days ago

  • Status changed from Solution validée to 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)

#10

Updated by Transition automatique 6 days ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF