http_client docstring does not say that timeout, verify and proxies only configure the default session
Avaliação
- Dificuldade
- 1/5
- Tempo estimado
- Menos de uma hora
- Facilidade para iniciantes
- 25/100
- Tipo de issue
- Documentação
- Clareza
- Claramente especificada
- Status de atividade
- Estagnada
- Stack de tecnologia
- python
- Domínio
- documentation
Direção de pesquisa
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.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- Python
- Estrelas
- 971
- Forks
- 223
- Merge médio
- 3h 6min
- PRs com merge (30d)
- 4
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de AzureAD/microsoft-authentication-library-for-python
-
AzureSDK bug P1
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 48/100
-
[Bug] The built-in retry never retries the token POST: HTTPAdapter(max_retries=1) keeps urllib3's default allowed_methodsTalvez já em andamento @HardMax71 assumiu há 44 dias. Abertabug untriaged
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 35/100
AzureAD/microsoft-authentication-library-for-python#957 · 3 comentários ·
-
public-client untriaged
Dificuldade 5/5 Mais de uma semana Facilidade para iniciantes 25/100
-
[Bug] Interactive auth callback hangs forever when an idle connection reaches AuthCodeReceiver firstTalvez já em andamento @Thanatos9404 assumiu há 3 dias. Abertabug public-client
Dificuldade 4/5 1-2 dias Facilidade para iniciantes 48/100
AzureAD/microsoft-authentication-library-for-python#950 · 2 comentários · 3 reações ·
-
needs attention public-client Similar-Issue untriaged
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 43/100
AzureAD/microsoft-authentication-library-for-python#908 · 3 comentários ·
Todas as issues de AzureAD/microsoft-authentication-library-for-python
Issues semelhantes
-
Dificuldade 1/5 Menos de uma hora Facilidade para iniciantes 65/100
521xueweihan/HelloGitHub#4025 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 85/100
aws-samples/sample-per-model-bedrock#321 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 1/5 1-3 horas Facilidade para iniciantes 68/100
FavioVazquez/showtime#101 ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
koush/ha_scrypted#81 ·
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 82/100
google/osv-scanner-action#147 ·