nodeSolidServer / nodeSolidServer/node-solid-server
NSS try to create path, filename containing invalid windows characters like colon
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.8k
- Forks
- 308
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
@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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne in resource-mapper.js und verfolge, wie die angeforderte URI zu einem Windows-Dateisystempfad wird. Reproduziere den chat-channel PUT unter Windows mit dem Pfad, der einen Doppelpunkt enthält, und ermittle anschließend, ob die Verarbeitung betriebssystemabhängig sein sollte, während gültige URI-Pfade kompatibel bleiben. Als erledigt gilt die Aufgabe, wenn die Anfrage nicht mehr aufgrund von unter Windows verbotenen Pfadzeichen fehlschlägt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js
- Bereich
- backend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 35/100