plenary.curl breaks headers if _ is required in header name
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 45/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- lua
- Área
- networking
Línea de trabajo
Start with lua/plenary/curl.lua at the linked line 101 and trace how request headers are transformed. Reproduce the request using a header such as CUSTOM_AUTH_HEADER, then determine whether the completed change should preserve underscores or provide an override, with the request accepted by the target service as the result.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
I have a service that requires a header with an _ in it's name. (Think CUSTOM_AUTH_HEADER or smth like that). I wanted to test it through a library that uses plenary.curl, and found that the request returns 404: upon closer inspection it turned out that the culprit is this line:
https://github.com/nvim-lua/plenary.nvim/blob/master/lua/plenary/curl.lua#L101
turning my header into CUSTOM-AUTH-HEADER, therefore making the request invalid. Is there a way to override this? As far as I know the HTTP spec doesn't enforce - over _, so I'd like to have a solution (or at least know why this mapping was made).
Thanks
- Lenguaje dominante
- Lua
- Estrellas
- 3.5k
- Forks
- 341
- 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 nvim-lua/plenary.nvim
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
nvim-lua/plenary.nvim#682 ·
-
uniform usage of `os.getenv`Posiblemente ocupada @rakotomandimby la tomó hace 124 días. Abierto
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
nvim-lua/plenary.nvim#680 ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 28/100
nvim-lua/plenary.nvim#672 · 1 comentario ·
-
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
nvim-lua/plenary.nvim#671 · 1 comentario ·
-
Windows: path separator issuesAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 25/100
nvim-lua/plenary.nvim#670 ·
Todos los issues de nvim-lua/plenary.nvim
Issues similares
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 82/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 70/100
-
documentation good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
InauguralSystems/EigenLua#60 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 1/5 1-3 horas Aptitud para principiantes 85/100
bradcypert/plum#265 ·
Los mantenedores suelen responder en 1 día
-
Data Correction Forever
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
Los mantenedores suelen responder en 1 día