jazzband/django-silk

raise codec error in execute_sql

Open

#157 aberto em 6 de jan. de 2017

Ver no GitHub
 (1 comment) (0 reactions) (0 assignees)Python (3.958 stars) (319 forks)batch import
bughelp wanted

Description

The statement would raise codec error when "params" contained non ASCII strings at sql.py: sql_query = q % params It's better to use django's force_text processing to handle it.

Guia do colaborador

Pilha de tecnologia
pythonsql
Domain
backenddatabase
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.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
under 1 hour
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
stale
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
clear
Prerequisites
PythonDjango
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
85
Direção de pesquisa
The bug is in sql.py where `sql query = q % params` causes codec errors with non ASCII strings. The fix is to use Django's `django.utils.encoding.force text` to safely handle the params. Look at the existing codebase for similar patterns and add a test case with non ASCII characters to verify the fix.