PythonCharmers/python-future

urllib.parse.unquote may return bytes

Open

#431 aberto em 21 de jan. de 2019

Ver no GitHub
 (1 comment) (0 reactions) (1 assignee)Python (327 forks)batch import
0.19help wanted

Métricas do repositório

Stars
 (1.172 stars)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

In Py3, unquote does not accept bytes: the first line if '%' not in string would throw exception.

However, in Py2, that line will not throw exception and perform the checking.

When there's no %, the return value is the argument. That is, when the argument type is bytes, the return value is bytes, too.

Code snippet excerption from src/future/backports/urllib/parse.py#L515:

def unquote(string):
  if '%' not in string:
    string.split
    return string      # bytes or unicode, depends on argument; Py3 never reach here
  ...
  return ''.join(res)  # unicode

Guia do colaborador

Direção de pesquisa
O problema está em src/future/backports/urllib/parse.py na linha 515. A função unquote deve retornar str tanto no Python 2 quanto no 3. A correção deve garantir que, quando a entrada for bytes, ela seja decodificada ou convertida para str antes do processamento. Considere adicionar uma verificação para o tipo bytes e converter usando .decode('utf 8') ou similar. O código atual retorna bytes se não houver '%' na string, o que é inconsistente. Verifique a implementação do Python 3 para referência.
Pilha de tecnologia
python
Domain
backend
Tipo Issue
Bug
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
3
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Antigo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Basic understanding of Python 2/3 string handlingFamiliarity with urllib.parse
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
60