Projet

Général

Profil

Development #58340

backend ldap : encore du code de compat python 2 à virer

Ajouté par Paul Marillonnet il y a plus de 2 ans. Mis à jour il y a plus de 2 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Catégorie:
-
Version cible:
-
Début:
03 novembre 2021
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:
Non

Description

je vais virer ça en amont de #58151


Fichiers


Demandes liées

Lié à Authentic 2 - Development #58151: préciser sur un échec d'authentification dans le journal que ça vient d'un annuaire LDAP downFermé25 octobre 2021

Actions

Révisions associées

Révision f8914100 (diff)
Ajouté par Paul Marillonnet il y a plus de 2 ans

ldap: discard obsolete python2 compat code (#58340)

Historique

#1

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Lié à Development #58151: préciser sur un échec d'authentification dans le journal que ça vient d'un annuaire LDAP down ajouté
#2

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Quelque chose comme ça, à vue de nez.

#3

Mis à jour par Thomas Noël il y a plus de 2 ans

Tu supposes aussi que Authentic devient systématiquement dépendant du module ldap ? (ça me dérange pas du tout, mais je pose la question ; si ça se trouve on a d'autres dépendances sur ldap par ailleurs)

#4

Mis à jour par Paul Marillonnet il y a plus de 2 ans

Thomas Noël a écrit :

Tu supposes aussi que Authentic devient systématiquement dépendant du module ldap ? (ça me dérange pas du tout, mais je pose la question ; si ça se trouve on a d'autres dépendances sur ldap par ailleurs)

Je ne sais pas quand on s’est imaginé qu’il y aurait a2 sans ce module, mais cette réflexion n’a pas été reportée dans l’empaquetage : le paquet python3-ldap est présent dans les dépendances debian et le module python-ldap l’est aussi dans les install_requires du setup.py.

#5

Mis à jour par Thomas Noël il y a plus de 2 ans

Paul Marillonnet a écrit :

Thomas Noël a écrit :

Tu supposes aussi que Authentic devient systématiquement dépendant du module ldap ? (ça me dérange pas du tout, mais je pose la question ; si ça se trouve on a d'autres dépendances sur ldap par ailleurs)

Je ne sais pas quand on s’est imaginé qu’il y aurait a2 sans ce module

Dans notre jeunesse. Mais de toute façon y'a une dépendance obligatoire aujourd'hui dans src/authentic2/ldap_utils.py qui est utilisé inconditionnellement par src/authentic2/backends/ldap_backend.py (from authentic2.ldap_utils ...) donc j'ai pas de soucis du tout. Je faisais juste une remarque, genre le gars qui fait une relecture.

#6

Mis à jour par Thomas Noël il y a plus de 2 ans

  • Statut changé de Solution proposée à Solution validée
#7

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Statut changé de Solution validée à Résolu (à déployer)
commit f8914100ff6c020bacfa6e221fb29f32186b3036
Author: Paul Marillonnet <pmarillonnet@entrouvert.com>
Date:   Wed Nov 3 11:21:08 2021 +0100

    ldap: discard obsolete python2 compat code (#58340)
#8

Mis à jour par Paul Marillonnet il y a plus de 2 ans

  • Statut changé de Résolu (à déployer) à Solution déployée

Formats disponibles : Atom PDF