nodeSolidServer / nodeSolidServer/node-solid-server
Incorrect HTTP method in response message and duplicate Link header when creating a container
Offen
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.8k
- Forks
- 308
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
curl -iX POST -H'Link: <http://www.w3.org/ns/ldp#BasicContainer>; rel="type"' https://example.org/
HTTP/2 415
link: <.acl>; rel="acl", <.meta>; rel="describedBy", <http://www.w3.org/ns/ldp#Container>; rel="type", <http://www.w3.org/ns/ldp#BasicContainer>; rel="type"
PUT request require a valid content type via the Content-Type header
Note POST request but PUT in response message.
curl -iX POST -H'Content-Type: text/turtle' -H'Link: <http://www.w3.org/ns/ldp#BasicContainer>; rel="type"' https://example.org/
HTTP/2 201
location: /c4db39a0-2bc3-11ea-b34b-d1f743b7eb53/
link: <.acl>; rel="acl", <.meta>; rel="describedBy", <http://www.w3.org/ns/ldp#Container>; rel="type", <http://www.w3.org/ns/ldp#BasicContainer>; rel="type", <http://www.w3.org/ns/ldp#BasicContainer>; rel="type"
Created
Note duplicate LDP-BC rel=type. Same as https://github.com/solid/node-solid-server/issues/426
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 damit, beide POST-Anfragen aus dem Issue zu reproduzieren, und verfolge die HTTP-Verarbeitung zur Container-Erstellung im NodeJS-Server. Bestätige, dass die Fehlermeldung POST statt PUT nennt und dass die Antwort den LDP BasicContainer-Typ im Link-Header nur einmal ausgibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js
- Bereich
- api, backend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Größtenteils klar
- Anfängerfreundlichkeit
- 45/100