http_client docstring does not say that timeout, verify and proxies only configure the default session
Évaluation
- Difficulté
- 1/5
- Temps estimé
- Moins d'une heure
- Accessibilité débutants
- 25/100
- Type d'issue
- Documentation
- Clarté
- Clairement spécifiée
- Activité
- À l'abandon
- Stack technique
- python
- Domaine
- documentation
Piste de recherche
Read the http_client paragraph in msal/application.py at lines 496–502, then compare it with the timeout, verify and proxies parameter notes at lines 508, 513 and 518. Add one sentence pointing readers to those parameters, and check that the paragraph makes clear they configure only the default session. A linked open pull request already addresses this issue.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
The http_client paragraph of ClientApplication.__init__ (msal/application.py:496-502 on dev) explains retries but says nothing about timeout, verify and proxies. Those three parameters each carry "This does not apply if you have chosen to pass your own Http client" (lines 508, 513, 518), so the information exists, but only if you read those three entries; someone reading the http_client docs to decide whether to pass one does not see it.
The behaviour itself is as documented: timeout is applied with functools.partial on the session MSAL builds (application.py:702-713) and a custom client is wrapped in ThrottledHttpClient untouched. Checked on 1.38.0 with a recording fake client: its request receives no timeout kwarg, while the default session's request is a partial with timeout=5.
Proposal: one sentence in the http_client paragraph pointing at the three parameters. PR follows.
- Langage dominant
- Python
- Étoiles
- 971
- Forks
- 223
- Merge moyen
- 13 h 48 min
- PR mergées (30 j)
- 2
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de AzureAD/microsoft-authentication-library-for-python
-
needs attention untriaged
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
-
AzureSDK bug P1
Difficulté 4/5 3-5 jours Accessibilité débutants 48/100
-
[Bug] The built-in retry never retries the token POST: HTTPAdapter(max_retries=1) keeps urllib3's default allowed_methodsPeut-être pris @HardMax71 l’a pris il y a 45 jours. Ouvertebug untriaged
Difficulté 2/5 1-3 heures Accessibilité débutants 35/100
AzureAD/microsoft-authentication-library-for-python#957 · 3 commentaires ·
-
public-client untriaged
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 25/100
-
[Bug] Interactive auth callback hangs forever when an idle connection reaches AuthCodeReceiver firstPeut-être pris @Thanatos9404 l’a pris il y a 4 jours. Ouvertebug public-client
Difficulté 4/5 1-2 jours Accessibilité débutants 48/100
AzureAD/microsoft-authentication-library-for-python#950 · 2 commentaires · 3 réactions ·
Toutes les issues de AzureAD/microsoft-authentication-library-for-python
Issues similaires
-
Difficulté 1/5 1-3 heures Accessibilité débutants 66/100
AlexsLemonade/scpca-docs#518 ·
-
bug
Difficulté 1/5 Moins d'une heure Accessibilité débutants 95/100
rocketride-org/rocketride-server#2705 · 1 commentaire ·
Les mainteneurs répondent en général sous 1 jour
-
documentation
Difficulté 2/5 1-3 heures Accessibilité débutants 85/100
Metabuilder-Labs/tokenjam#786 ·
Les mainteneurs répondent en général sous 1 jour
-
prototype(verifier_token): map disallowed warning to contraindicated before composite precedence checkPeut-être pris @drk-ram l’a pris aujourd’hui. Ouverte
Difficulté 2/5 1-3 heures Accessibilité débutants 88/100
agentrust-io/trace-spec#582 ·
Les mainteneurs répondent en général sous 1 jour
-
area:cli priority:low refactor size:small
Difficulté 1/5 Moins d'une heure Accessibilité débutants 88/100
RonaldHensbergen/composable-data-stack#906 ·
Les mainteneurs répondent en général sous 1 jour