nodeSolidServer / nodeSolidServer/node-solid-server

POST not allowed for auxiliary resources

オープン
#1,528 コメント 1 件 リアクション 0 件 担当者 1 名 GitHub で見る

@bourgeoa がすでに取り組んでいます。

2020年11月28日 から。

主要言語
JavaScript
スター
1.8k
フォーク
308
PR マージ指標
30日以内にマージされた PR はありません

説明

@csarven @michielbdejong @jeff-zucker

When a POST method request with the Slug header targets an auxiliary resource, the server MUST respond with the 403 status code and response body describing the error. [Source]

This is taken from specification.
It does not seem to fit well with : PR #1527

When a DELETE request targets storage’s root container or its associated ACL resource, the server MUST respond with the 405 status code. Server MUST exclude the DELETE method in the HTTP response header Allow in response to safe method requests [RFC7231]. [Source]

Why not use 405 for both and exclude the method from the headers response ? Is there a conflict with http specifications ?

When I look at it from the error code side it seems also easier to understand for Solid users and devs :
This is certainly a simplification.

  • 400 do not exist (resource, content-type, ...)
  • 401 user has no authorization (not logging ...)
  • 403 user is not allowed ( relates to user permission...)
  • 405 method is not allowed in this request (DELETE for root, POST for auxiliary resources, ...)
  • 409 conflict (cannot delete container with resources, cannot inserts with patch when deletes fails, cannot create a resource with the same name than an existing container, ...)
  • 415 exist but unknown (content-type mime-type is unknown, ...)

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。