ParseResultBytes.unsplit() raises AttributeError for a reference without an authority
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 84/100
Línea de trabajo
Empieza en parseresult.py, en la propiedad de bytes authority, alrededor de la línea 409, y compara cómo gestiona la ausencia de una authority con las propiedades de bytes protegidas que se mencionan en el issue. Añade una prueba de regresión para ParseResultBytes.from_string(b"/a").unsplit() y comprueba que hace el recorrido de ida y vuelta sin lanzar una excepción; el issue no indica ningún archivo de pruebas, así que localiza primero las pruebas de parseresult existentes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
>>> from rfc3986.parseresult import ParseResult, ParseResultBytes
>>> ParseResult.from_string("/a").unsplit()
'/a'
>>> ParseResultBytes.from_string(b"/a").unsplit()
AttributeError: 'NoneType' object has no attribute 'encode'
Any reference without an authority (/a, a, mailto:x, ?q) crashes the bytes variant, while the str variant round-trips. Reproduced on 2.0.0 (main 3bd3832).
The traceback: unsplit calls copy_with(lazy_normalize=False), which calls _generate_authority, which reads the authority property, and that property does self.reference.authority.encode(self.encoding) (parseresult.py:409) without checking for None. The other bytes properties (scheme, path, …) are guarded the same way authority should be.
Happy to send a PR with a test if you would take one.
- Lenguaje dominante
- Python
- Estrellas
- 189
- Forks
- 43
- Merge medio
- 12 h 14 min
- PR fusionados (30 d)
- 2
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de python-hyper/rfc3986
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
python-hyper/rfc3986#164 ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 45/100
python-hyper/rfc3986#138 · 1 comentario ·
-
sdist is missing `tox.ini`Quizá libre de nuevo Un pull request para esta issue se cerró sin fusionarse. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 55/100
python-hyper/rfc3986#129 ·
-
wrong parseAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 38/100
python-hyper/rfc3986#128 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 35/100
python-hyper/rfc3986#123 · 2 comentarios ·
Todos los issues de python-hyper/rfc3986
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Arekkazu/sgpmp-backend#744 ·
Los mantenedores suelen responder en 1 día
-
Review backlog (security): master-branch remote guard trusts unbound provenance package fieldAbiertoagent/reviewer
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
projectbluefin/utah-packages#634 ·
Los mantenedores suelen responder en 1 día
-
enhancement good first issue help wanted
Dificultad 2/5 1-2 días Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
home-assistant/core#188613 ·
Los mantenedores suelen responder en 1 día
-
WebsiteAbiertogood first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 91/100
TanbirRamim/open-source-radar#314 ·
Los mantenedores suelen responder en 1 día