nodeSolidServer / nodeSolidServer/node-solid-server

URI encoding for Windows OS

Abierto
#1,542 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Lenguaje dominante
JavaScript
Estrellas
1.8k
Forks
308
Métricas de merge de PR
Sin PR fusionados en 30 d

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 :

  1. remove the control in POST
  2. add the control in all write functions including PUT PATCH (filename and intermediate containers)

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 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.

Evaluación

Stack tecnológico
javascript, node.js
Área
backend
Tipo de issue
Error
Dificultad
4/5
Tiempo estimado
3-5 días
Estado de actividad
Estancado
Claridad
Bastante claro
Aptitud para principiantes
35/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.