Double encoding of anchor tag href
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 52/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript
- Área
- web-dev
Línea de trabajo
Comienza con cjs/html/anchor-element.js en la implementación enlazada y compárala con el issue y fix relacionados de linkedom. Reproduce el ejemplo usando un href que contenga %20 y verifica después que leer el anchor conserve la codificación existente en lugar de producir %2520.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Hey,
I've been using your amazing library to extract links from DOM content in a service worker and noticed that achor tags which, for example, lead to some file which has already encoded space characters in the URL (www.test.com/path%20to%20some%20file.pdf) end up with a broken href because the % character is encoded again (www.test.com/path%2520to%2520some%2520file.pdf).
I have a workaround by just calling decodeURIComponent on the href attribute before processing it but I guess that is not the intended behavior and shouldn't be like that.
I am sure that it's related to https://github.com/WebReflection/linkedom/issues/49 and the fix for it here https://github.com/WebReflection/linkedom/blob/5b31c583c79423d97fa1982d9a30a8f0a0982485/cjs/html/anchor-element.js#L18
I would guess and hope it's not a complicated fix.
Thanks a lot for your work!
- Lenguaje dominante
- HTML
- Estrellas
- 2.1k
- Forks
- 108
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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 WebReflection/linkedom
-
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
WebReflection/linkedom#329 · 5 comentarios ·
-
`el.style.property = ''` and `e.style.setProperty('property', '')` should delete the propertyAbierto
Dificultad 3/5 1-2 días Aptitud para principiantes 52/100
WebReflection/linkedom#327 · 1 comentario ·
-
Dificultad 3/5 1-2 días Aptitud para principiantes 48/100
WebReflection/linkedom#326 · 5 comentarios ·
-
Build Apps For AI Agent.Abierto
Dificultad 5/5 Más de una semana Aptitud para principiantes 15/100
WebReflection/linkedom#321 · 1 comentario ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
WebReflection/linkedom#318 · 6 comentarios ·
Todos los issues de WebReflection/linkedom
Issues similares
-
agent-reported area/browser bug good first issue hacktoberfest help wanted P1
Dificultad 2/5 Medio día Aptitud para principiantes 78/100
Los mantenedores suelen responder en 2 días
-
Create User page w/ contentAbiertofull-stack
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
kir-dev/pek-infinity#107 ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 86/100
Los mantenedores suelen responder en 1 día
-
accessibility area:ai-suggestions bug good first issue P2
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
uttrflow/uttrflow-swift#1893 ·
Los mantenedores suelen responder en 1 día
-
[Bug] : Main navigation "Upcoming Events" link navigates to /events/ without query parameterAbiertobug
Dificultad 2/5 1-3 horas Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día