minor performance optimization concerning location

Aberta
  • Algo errado?
#136 0 comentários 0 reações 0 responsáveis Ver no GitHub

Ninguém assumiu esta issue ainda.

Avaliação

Dificuldade
1/5
Tempo estimado
Menos de uma hora
Facilidade para iniciantes
45/100
Tipo de issue
Refatoração
Clareza
Claramente especificada
Status de atividade
Estagnada
Stack de tecnologia
python
Domínio
api, backend

Direção de pesquisa

Abra client.py na linha 450, em get_response, e inspecione como location é obtido em relação a follow_redirect. Mova essa consulta para dentro da condicional, para que ela ocorra somente quando os redirecionamentos forem seguidos, e então verifique se o comportamento existente de response e dos redirecionamentos permanece inalterado.

Escrita pelo modelo de indexação a partir do texto da issue.

Descrição

It's probably not going to matter all that much in practice, but I notice that in client.py line 450 (get_response) the location is taken from the headers unconditionally, whereas it is only needed if follow_redirect is True, so could move into the 'if' block.

Linguagem predominante
Python
Estrelas
402
Forks
69
Nenhum PR com merge em 30d

Preparar o ambiente

Este projeto não oferece contêiner de desenvolvimento, Dockerfile nem guia de contribuição, então a configuração fica por sua conta: comece pelo README e veja nosso guia da primeira contribuição para os passos gerais.

Primeiros passos

  1. Leia a issue inteira e depois o guia de contribuição do projeto.
  2. Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
  3. Faça um fork do repositório e trabalhe em uma branch.
  4. Abra um pull request que referencie o número da issue.

Mais de benoitc/restkit

Todas as issues de benoitc/restkit

Issues semelhantes

Mais issues de Python

Receba novas issues na sua caixa de entrada

Um resumo curto de issues do GitHub para quem está começando.