python / python/cpython

`urllib.parse.parse_qsl` is accepting illegal characters

Abierto
#138,284 2 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

stdlib type-bug
Lenguaje dominante
Python
Estrellas
77.2k
Forks
36k
Métricas de merge de PR
Métricas de PR pendientes

Descripción

Bug report

Bug description:

urllib.parse.parse_qsl parses query strings containing the ^ and ` characters, even though these are not valid query characters under RFC 3986.

Observed behaviour:

parse_qsl('foo=^', strict_parsing=True)
# [('foo', '^')]

parse_qsl('bar=`', strict_parsing=True)
# [('bar', '`')]

Expected behaviour:
According to RFC 3986, both ^ and ` must be percent-encoded if used in a URI. However, parse_qsl accepts them as-is without raising an error or warning. This could lead to applications treating invalid URLs as valid.

Detailed Code:

import sys
import platform
from urllib.parse import parse_qsl


def test_parse_qsl(query):
    try:
        result = parse_qsl(query, strict_parsing=True)
        print(f"Query: {query!r} -> Parsed: {result}")
    except ValueError as e:
        print(f"Query: {query!r} -> Error: {e}")


# Test invalid query strings
test_parse_qsl("foo=^")
test_parse_qsl("bar=`")

# System information
os_name = platform.system()
os_release = platform.release()
python_impl = platform.python_implementation() 
python_version = sys.version.split()[0]
python_compiler = platform.python_compiler() 

print("\n--- System Information ---")
print(f"Python Implementation : {python_impl}")
print(f"Python Version : {python_version}")
print(f"Python Compiler       : {python_compiler}")
print(f"Operating System : {os_name} {os_release}")
print(f"Machine         : {platform.machine()}")

# Output:
# Query: 'foo=^' -> Parsed: [('foo', '^')]
# Query: 'bar=`' -> Parsed: [('bar', '`')]

# --- System Information ---
# Python Implementation : CPython
# Python Version : 3.13.1
# Python Compiler       : GCC 14.2.0
# Operating System : Linux 4.14.174
# Machine         : x86_64

References:

  1. Section 2.2 Reserved Characters: https://datatracker.ietf.org/doc/html/rfc3986#section-2.2

  2. Section 2.3 Unreserved Characters: https://datatracker.ietf.org/doc/html/rfc3986#section-2.3

CPython versions tested on:

3.13

Operating systems tested on:

Linux

Linked PRs
  • gh-138291

Guía de contribución

Abrir la guía de contribución

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 en el punto de entrada urllib.parse.parse_qsl y compara su comportamiento actual con las secciones 2.2 y 2.3 de RFC 3986. Revisa las pruebas existentes de urllib.parse y verifica después que las consultas que contienen ^ y ` sin codificar se gestionen según lo especificado y que la cobertura de regresión capture el comportamiento esperado.

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

Evaluación

Stack tecnológico
python
Área
networking
Tipo de issue
Error
Dificultad
3/5
Tiempo estimado
1-2 días
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
25/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.