python / python/cpython

Issues in ipaddress.collapse_addresses()

Ouverte
#149,722 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

3.13 3.14 3.15 3.16 stdlib type-bug type-feature
Langage dominant
Python
Étoiles
77.2k
Forks
35.9k
Métriques de merge des PR
Métriques de PR en attente

Description

Bug report

The following issues was discovered during reviewing #141842:

  • ipaddress.collapse_addresses() supports not only IP networks, but also IP addresses and a mix of them. There are dedicated tests, but this is not documented.
  • ipaddress.collapse_addresses() contains code for supporting also IP interfaces, but it always was broken (it would work if interfaces were not subclasses of addresses). The test for mix of IP addresses and IP interfaces is mislabeled (it says a mix of addresses and networks) and partially commented out (it only tests addresses now, which is mostly a duplicate of other test).
  • While TypeError is raised immediately for IP addresses of different versions and IP networks of different versions, it is delayed until the start of iteration for a mix of IP addresses and IP networks with addresses and networks having different versions.
  • For objects other that IP objects you can get AttributeError instead of TypeError.

My plan for a fix:

  • Document support of IP addresses.
  • Explicitly forbid IP interfaces.
  • Always raise TypeError for invalid types (the code is simpler this way).
  • Add tests for corner cases.

For new version only:

  • Implement support of IP interfaces and document it as a new feature.
  • Either raise TypeError immediately for IP objects of different versions, or remove redundant explicit checks, they will be raised implicitly in comparisons.
Linked PRs
  • gh-149723

Guide de contribution

Ouvrir le guide de contribution

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Piste de recherche

Commencez par le point d’entrée ipaddress.collapse_addresses() et les tests dédiés existants mentionnés dans le rapport. Examinez la PR liée gh-149723 avant de modifier le comportement. Le travail est considéré comme terminé lorsque les adresses IP prises en charge sont documentées, que le traitement des interfaces IP est défini, que TypeError est systématiquement levée pour les entrées invalides et que des tests de cas limites sont ajoutés.

Rédigé par le modèle d'indexation à partir du texte de l'issue.

Évaluation

Stack technique
python
Domaine
networking
Type d'issue
Bug
Difficulté
4/5
Temps estimé
3-5 jours
Activité
À l'abandon
Clarté
Plutôt claire
Accessibilité débutants
25/100

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.