http_client docstring does not say that timeout, verify and proxies only configure the default session
Bewertung
- Schwierigkeit
- 1/5
- Geschätzter Aufwand
- Unter einer Stunde
- Anfängerfreundlichkeit
- 25/100
- Issue-Typ
- Dokumentation
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- python
- Bereich
- documentation
Rechercherichtung
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.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
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.
- Vorherrschende Sprache
- Python
- Sterne
- 971
- Forks
- 223
- Ø Merge
- 13 Std. 48 Min.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus AzureAD/microsoft-authentication-library-for-python
-
needs attention untriaged
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
-
AzureSDK bug P1
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
-
[Bug] The built-in retry never retries the token POST: HTTPAdapter(max_retries=1) keeps urllib3's default allowed_methodsEvtl. vergeben @HardMax71 hat das vor 46 Tagen übernommen. Offenbug untriaged
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 35/100
AzureAD/microsoft-authentication-library-for-python#957 · 3 Kommentare ·
-
public-client untriaged
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 25/100
-
[Bug] Interactive auth callback hangs forever when an idle connection reaches AuthCodeReceiver firstEvtl. vergeben @Thanatos9404 hat das vor 5 Tagen übernommen. Offenbug public-client
Schwierigkeit 4/5 1-2 Tage Anfängerfreundlichkeit 48/100
AzureAD/microsoft-authentication-library-for-python#950 · 2 Kommentare · 3 Reaktionen ·
Alle Issues in AzureAD/microsoft-authentication-library-for-python
Ähnliche Issues
-
wr.dynamodb.read_items returns [{}] for a single partition value that does not existEvtl. vergeben @auraseb hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 84/100
aws/aws-sdk-pandas#3520 ·
Maintainer antworten meist innerhalb von 2 Tagen
-
requires triage typing
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
sqlalchemy/sqlalchemy#13704 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Edit tool converts CRLF files to LFEvtl. vergeben @RaphaelFakhri hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
huggingface/tau#774 ·
Maintainer antworten meist innerhalb von 1 Tag
-
show ignores the row limit when a column or table name contains "limit"Evtl. vergeben @RaphaelFakhri hat das heute übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 88/100
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
linebender/resvg#1179 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 2 Tagen