Hacktoberfest 2026 : les issues que les mainteneurs ont marquées pour octobre, ouvertes et accessibles aux débutants. Parcourir les issues Hacktoberfest

Cache requests to .well-known/openid_configuration/

Ouverte Adaptée aux débutants
  • Quelque chose ne va pas ?
#52 0 commentaires 0 réactions 0 personnes assignées Voir sur GitHub

Personne n'a encore pris cette issue.

Évaluation

Difficulté
2/5
Temps estimé
1-3 heures
Accessibilité débutants
68/100
Type d'issue
Fonctionnalité
Clarté
Clairement spécifiée
Activité
À l'abandon
Stack technique
python
Domaine
api, authentication

Piste de recherche

Commencez par get_discovery_doc et le chemin de construction de AuthClient décrit dans l’issue, puis examinez comment le document de découverte est actuellement demandé et attaché à la Session. Confirmez que les constructions répétées de AuthClient réutilisent les résultats de découverte mis en cache sans nécessiter de session, et vérifiez que la suite de tests existante couvre ce comportement.

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

Description

(related to https://github.com/intuit/oauth-pythonclient/issues/18)

Summary

.well-known/openid_configuration/ is a stable resource that is not expected to change between requests. If a caller constructs an AuthClient on every request (which I presume is quite common), then they will hit this endpoint, incurring latency costs and potential for failure. Given the latency on this endpoint, I would imagine these duplicate requests are not cheap to Intuit either.

This redundancy exacerbates issues like https://github.com/intuit/oauth-pythonclient/issues/47,

Solution

Currently get_discovery_doc is called each time an AuthClient is constructed, and attached to the Session extended by AuthClient

I would propose instead:

  • Do not call get_discovery_doc with a session arg -- it's a static resource that you can access without any auth.
  • Annotate get_discovery_doc with @lru_cache (docs) to cache results in-memory for each set of arguments. This is a python builtin.

Happy to contribute a PR for this if interested. I imagine this will provide pretty major cost savings to both Intuit (due to dramatic reduction in duplicate requests) and callers (due to saved compute and less network congestion)

@robert-mings If contributed, would you be open to something like this?

Langage dominant
Python
Étoiles
86
Forks
60
Aucune PR mergée en 30 j

Préparer son environnement

Ce projet ne fournit ni conteneur de développement, ni Dockerfile, ni guide de contribution : l'installation est à votre charge. Commencez par son README, et consultez notre guide de la première contribution pour les étapes générales.

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 intuit/oauth-pythonclient

Toutes les issues de intuit/oauth-pythonclient

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.