nodeSolidServer / nodeSolidServer/node-solid-server
NSS try to create path, filename containing invalid windows characters like colon
Ninguém assumiu esta issue ainda.
- Linguagem predominante
- JavaScript
- Estrelas
- 1.8k
- Forks
- 308
- Métricas de merge de PRs
- Nenhum PR com merge em 30d
Descrição
@timbl @michielbdejong @RubenVerborgh @highlevellogic
Trying NSS on a windows pod https://bourgeoa.solid.hll.nu:7443 . Tested the chat with me from OS solid
The function returns a PUT error 500
FAILED to save new chat channel at: https://bourgeoa.solid.hll.nu:7443/IndividualChats/bourgeoa.solid.hll.nu:7443/index.ttl : Web error: 500 (Internal Server Error) on PUT of https://bourgeoa.solid.hll.nu:7443/IndividualChats/bourgeoa.solid.hll.nu:7443/index.ttl
This is due to the colon : in the folderPath
To ensure solid compatibility can we forbid the windows forbidden characters in pathName like \/:*%"<>| ?
This could be done in resource-mapper.js
Should it be OS dependent to allow any file OS file be URI capable.
index:name.ttl is valid POSIX file and URI, but invalid WINDOWS
Guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Direção de pesquisa
Comece em resource-mapper.js e rastreie como a URI solicitada se transforma em um caminho do sistema de arquivos do Windows. Reproduza o PUT de chat-channel no Windows com o caminho que contém dois-pontos e, em seguida, determine se o tratamento deve depender do sistema operacional, mantendo a compatibilidade com caminhos URI válidos. A tarefa estará concluída quando a solicitação não falhar mais por causa de caracteres de caminho proibidos pelo Windows.
Escrita pelo modelo de indexação a partir do texto da issue.
Avaliação
- Stack de tecnologia
- javascript, node.js
- Domínio
- backend
- Tipo de issue
- Bug
- Dificuldade
- 3/5
- Tempo estimado
- 1-2 dias
- Status de atividade
- Estagnada
- Clareza
- Razoavelmente clara
- Facilidade para iniciantes
- 35/100