MagicStack / MagicStack/asyncpg

.pgpass file support handles backslash escapes incorrectly; should de-escape values

Abierto
#1,249 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
Python
Estrellas
8.1k
Forks
468
Métricas de merge de PR
Sin PR fusionados en 30 d

Descripción

https://www.postgresql.org/docs/current/libpq-pgpass.html isn't fully explicit about how to handle backslash escapes in the .pgpass file, however it's reasonably clear from the phrasing that backslashes serve only to escape backslashes and colons and therefore aren't considered part of the value they're used in. E.g., a .pgpass row of …

*:*:test\:db:test\\:password from pgpass with escapes

conveys:

  • a database name of test:db, not test\:db
  • a user name of test\, not test\\

I ran into this problem because I have a password of 4:K1…, encoded as 4\:K1…, and it works with psql but not asyncpg. Once I remove the backslash from .pgpass, it starts working with asyncpg but failing with psql.

I would've provided a failing test and even a code change, but I can't get python setup.py test or pip install -e .[test] to work in a run-of-the-mill Python 3.12 virtualenv on my run-of-the-mill MacOS.

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Comienza resolviendo el problema de configuración de las pruebas de Python 3.12 reportado para python setup.py test o pip install -e .[test], y luego localiza el manejo de .pgpass de asyncpg. Añade cobertura para dos puntos y barras invertidas escapados, y considera cerrado el issue cuando esos valores se desescapen de forma coherente con psql.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
postgresql, python
Área
databases
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
42/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.