URI encoding for Windows OS

Offen
#1,542 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Anfängerfreundlichkeit
35/100
Issue-Typ
Bug
Klarheit
Größtenteils klar
Aktivitätsstatus
Veraltet
Tech-Stack
javascript, node.js
Bereich
backend

Rechercherichtung

Beginne mit der Durchsicht von Commit 5bba6c931fe0b319b6b350b8338b6e144b0b67fa und der Schreibpfade für POST, PUT und PATCH und vergleiche anschließend den in solid/chat-pane#51 beschriebenen longChat-Fehler. Als erledigt gilt die Aufgabe, wenn die Behandlung von ':', '|' und '/' für POST, PUT, PATCH, Dateinamen und Zwischencontainer konsistent ist und das gewählte Verhalten dokumentiert oder durch die relevanten Tests abgedeckt wird.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

@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)
Vorherrschende Sprache
JavaScript
Sterne
1.8k
Forks
308
Keine gemergten PRs in 30 T.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus nodeSolidServer/node-solid-server

Alle Issues in nodeSolidServer/node-solid-server

Ähnliche Issues

Weitere Issues zu JavaScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.