URI encoding for Windows OS
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Error
- Claridad
- Bastante claro
- Estado de actividad
- Estancado
- Stack tecnológico
- javascript, node.js
- Área
- backend
Línea de trabajo
Comienza revisando el commit 5bba6c931fe0b319b6b350b8338b6e144b0b67fa y las rutas de escritura de POST, PUT y PATCH; después, compara el fallo de longChat descrito en solid/chat-pane#51. Se considera terminado cuando el tratamiento de ':', '|' y '/' sea coherente para POST, PUT, PATCH, nombres de archivo y contenedores intermedios, y el comportamiento elegido esté documentado o cubierto por las pruebas relevantes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
@timbl @angelo-v
The following PR in 2017 rejects the use of : | / in POST. I suppose it was to preserve the possibility to use NSS on windows.
https://github.com/solid/node-solid-server/commit/5bba6c931fe0b319b6b350b8338b6e144b0b67fa#diff-98a11be785b8100aab3a553eed0e3656a235f06463d2979fd08bafcdf503abdf
slug = decodeURIComponent(slug)
if (slug.match(/\/|\||:/)) {
throw error(400, 'The name of new file POSTed may not contain : | or /')
}
Using longChat in chat With Me encounters some problem because this control is not applied on PUT, when creating intermediate folders.
see https://github.com/solid/chat-pane/issues/51
I suppose we need some coherence, what is your position :
- remove the control in POST
- add the control in all write functions including PUT PATCH (filename and intermediate containers)
- Lenguaje dominante
- JavaScript
- Estrellas
- 1.8k
- Forks
- 308
- Métricas de merge de PR
- Sin PR fusionados en 30 d
Guía de contribución
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 nodeSolidServer/node-solid-server
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 72/100
nodeSolidServer/node-solid-server#1848 · 2 comentarios · 1 reacción ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
nodeSolidServer/node-solid-server#1841 · 2 comentarios ·
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
nodeSolidServer/node-solid-server#1147 ·
-
remove bootstrap dependency Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
nodeSolidServer/node-solid-server#1867 ·
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 45/100
nodeSolidServer/node-solid-server#1853 ·