Projet

Général

Profil

Tâche #11480

creole - Scénario #16098: Améliorer la validation des adresses IP/masque de sous-réseau dans Creole/Tiramisu

La validation actuelle du type "ip" n'est pas assez restrictive

Ajouté par Joël Cuissinat il y a presque 9 ans. Mis à jour il y a presque 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
Joël Cuissinat
Version cible:
Distribution EOLE - sprint 2016 19-21 - Equipe MENESR
Début:
04/05/2015
Echéance:
% réalisé:

100%

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

Description

Actuellement, il est possible de saisir un entier ou une adresse IP incomplète car "IPy" complète avec des "0" :

>>> import IPy
>>> IPy.IP('{0}/32'.format("1.1"))
IP('1.1.0.0')

Peut-être faudrait-il renforcer la validation actuelle en testant la présence des 3 points ?

Révisions associées

Révision 1c018102 (diff)
Ajouté par Joël Cuissinat il y a presque 8 ans

Test unitaire vérifiant la validation des IP saisies

Ref: #12858 @30m
Ref: #11480

Révision 6eb76f84
Ajouté par Joël Cuissinat il y a presque 8 ans

Merge remote-tracking branch 'lab/master'

Ref: #12858
Ref: #11480

Révision 24bb5b7e (diff)
Ajouté par Lionel Morin il y a presque 8 ans

Correction numéro de demande dans les commentaires (ref #11480 + ref #12858)

Historique

#1 Mis à jour par Joël Cuissinat il y a plus de 8 ans

  • Tracker changé de Anomalie à Tâche
  • Temps estimé mis à 2.00 h
  • Restant à faire (heures) mis à 2.0

A traiter en même temps que #12858.

#2 Mis à jour par Joël Cuissinat il y a plus de 8 ans

  • Tâche parente mis à #13918

#3 Mis à jour par Joël Cuissinat il y a presque 8 ans

  • Tâche parente changé de #13918 à #16098

#4 Mis à jour par Joël Cuissinat il y a presque 8 ans

  • Statut changé de Nouveau à En cours

#5 Mis à jour par Joël Cuissinat il y a presque 8 ans

Toujours d'actualité pour les types "ip" et "network".

Netmask aussi en fait mais il faut réfléchir pour contourner le test de cohérence ip/mask :

root@scribe:~# CreoleGet netmask_admin_eth0 
255.255.255
root@scribe:~# CreoleGet ip_admin_eth0 
1.1.1.0

#6 Mis à jour par Joël Cuissinat il y a presque 8 ans

  • Assigné à mis à Joël Cuissinat
  • % réalisé changé de 0 à 100
  • Restant à faire (heures) changé de 2.0 à 0.5

Cf. patch proposé dans #12858

#7 Mis à jour par Joël Cuissinat il y a presque 8 ans

  • Statut changé de En cours à Résolu

#8 Mis à jour par Joël Cuissinat il y a presque 8 ans

  • Description mis à jour (diff)
  • Restant à faire (heures) changé de 0.5 à 0.15

#9 Mis à jour par Lionel Morin il y a presque 8 ans

  • Statut changé de Résolu à Fermé
  • Restant à faire (heures) changé de 0.15 à 0.0

Application des tests unitaires => OK

Formats disponibles : Atom PDF