nodeSolidServer / nodeSolidServer/node-solid-server
URI encoding for Windows OS
Personne n'a encore pris cette issue.
- Langage dominant
- JavaScript
- Étoiles
- 1.8k
- Forks
- 308
- Métriques de merge des PR
- Aucune PR mergée en 30 j
Description
@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)
Guide de contribution
Ouvrir le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Piste de recherche
Commencez par examiner le commit 5bba6c931fe0b319b6b350b8338b6e144b0b67fa et les chemins d’écriture POST, PUT et PATCH, puis comparez l’échec de longChat décrit dans solid/chat-pane#51. Le travail est terminé lorsque le traitement de ':', '|' et '/' est cohérent pour POST, PUT, PATCH, les noms de fichiers et les conteneurs intermédiaires, et que le comportement choisi est documenté ou couvert par les tests pertinents.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- javascript, node.js
- Domaine
- backend
- Type d'issue
- Bug
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100