Tâche #22013
Scénario #21866: Traitement express MEN (46-48)
test dans certs.py pouvant provoquer des dysfonctionnements
Description
Dans la fonction de génération des certificats on trouve le test suivant :
# vérification de la valeur actuelle du ca # vérification du cn de la ca if os.path.isfile(ca_file): cmd = Popen(['/usr/bin/openssl', 'x509', '-in', ca_file, '-subject', '-noout'], stdout=PIPE) if cmd.wait() != 0: unlink(ca_file) prep_dir() else: res = cmd.communicate()[0] cn = res[res.rindex('CN=')+3:res.rindex('CN=')+15].strip() *if cn == str('self signed ') or cn == str('CA auto sign'):* unlink(ca_file) prep_dir()
Avec OpenSSL 1.0.2g le subject est :
subject= /C=FR/O=Ministere Education Nationale (MENESR)/OU=110 043 015/OU=ac-dijon/CN=CA-srvfog.cadoles.lan
Donc cn est égale a :
>>> res="subject= /C=FR/O=Ministere Education Nationale (MENESR)/OU=110 043 015/OU=ac-dijon/CN=CA-srvfog.cadoles.lan" >>> res[res.rindex('CN=')+3:res.rindex('CN=')+15].strip() 'CA-srvfog.ca'
Par contre avec OpenSSL 1.1.0f-2 (installé par FOG), le subject est :
subject=C = FR, O = Ministere Education Nationale (MENESR), OU = 110 043 015, OU = ac-dijon, CN = CA-srvfog.cadoles.lan
Le calcul de cn ne fonctionne plus :
>>> res="subject=C = FR, O = Ministere Education Nationale (MENESR), OU = 110 043 015, OU = ac-dijon, CN = CA-srvfog.cadoles.lan" >>> res[res.rindex('CN=')+3:res.rindex('CN=')+15].strip() Traceback (most recent call last): File "<stdin>", line 1, in <module> ValueError: substring not found
En regardant de plus prêt, je ne vois pas à quoi sert ce calcul et ce test.
C'est forcement toujours différent de 'self signed ' et 'CA auto sign'
Si on remonte dans l'historique, le commit creole:f684ca625509bd334b520b4576a32adfda7ba84a introduit ce test. Il a l'air de faire suite au passage d'openvpn à openssl pour générer les certificats creole:2b7ab94732f2425b0cd71cfb12653f36218d23a3
J'ai tendance à pense que ce calcul peu sûr n'est absolument plus nécessaire.
Révisions associées
Suppression du test sur les certificats autosignés (ref #22013)
Historique
#1 Mis à jour par Emmanuel GARETTE il y a plus de 6 ans
- Statut changé de Nouveau à En cours
- Assigné à mis à Emmanuel GARETTE
- Temps estimé mis à 1.00 h
- Restant à faire (heures) mis à 1.0
#2 Mis à jour par Emmanuel GARETTE il y a plus de 6 ans
- % réalisé changé de 0 à 100
- Restant à faire (heures) changé de 1.0 à 0.25
#3 Mis à jour par Scrum Master il y a plus de 6 ans
- Statut changé de En cours à Résolu
#4 Mis à jour par Benjamin Bohard il y a plus de 6 ans
- Restant à faire (heures) changé de 0.25 à 0.0
#5 Mis à jour par Benjamin Bohard il y a plus de 6 ans
- Statut changé de Résolu à Fermé