`inspect.getmembers_static` triggers `__getattribute__`

Ouverte
#99,673 1 commentaire 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

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

Piste de recherche

Commencez par l’implémentation de inspect.getmembers_static et le travail sur dir_static référencé par gh-55979. Reproduisez l’exemple de récursion, puis vérifiez que l’inspection des membres ne déclenche plus getattribute via dir ; c’est terminé lorsque le comportement signalé est corrigé sans modifier les limites documentées concernant les attributs créés dynamiquement.

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

Description

stdlib type-bug

Bug report

inspect.getmembers_static is documented as not triggering __getattribute__, but it can trigger it via dir. For example the following hits the recursion limit:

class A:
    def __getattribute__(self, name: str) -> Any:
        getmembers_static(self)
        return "all attrs exist!"

A().anything

(A workaround for examples like this one is to add the base cases required by dir to __getattribute__:

if name in ("__dict__", "__class__"):
    return object.__getattribute__(self, name)

(The __dict__ case is not always needed depending on the __dir__ implementation.))

Switching getmembers_static to use dir_static (gh-55979) would fix this problem, I believe. getmembers_static already documents that it can't find the dynamically created attributes, etc. that getattr_static can't find, so I don't think such a change would be breaking.

Your environment

  • CPython versions tested on: 3.11.0
  • Operating system and architecture: Linux-based, x86_64
Langage dominant
Python
Étoiles
77.2k
Forks
36k
Merge moyen
1 j 9 h
PR mergées (30 j)
558

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.

Autres issues de python/cpython

Toutes les issues de python/cpython

Issues similaires

Plus d'issues Python

Recevez les nouvelles issues par e-mail

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