Projet

Général

Profil

Tâche #22013

Scénario #21866: Traitement express MEN (46-48)

test dans certs.py pouvant provoquer des dysfonctionnements

Ajouté par Emmanuel GARETTE il y a plus de 6 ans. Mis à jour il y a plus de 6 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Début:
10/11/2017
Echéance:
% réalisé:

100%

Temps estimé:
1.00 h
Temps passé:
Restant à faire (heures):
0.0

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

Révision 4bcbd41b (diff)
Ajouté par Emmanuel GARETTE il y a plus de 6 ans

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é

Formats disponibles : Atom PDF