nodeSolidServer / nodeSolidServer/node-solid-server

:error in .acl causes all requests to error

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

まだ誰も着手していません。

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

説明

I might be wrong about this, but I think :error in an .acl should cause an error for only the group it refers to. However, if it is present in a .acl it will cause every request to return 500.

To reproduce enter /test/resources/accounts-acl/tim.localhost/group/test-folder/.acl and uncomment

#:errors
#    a acl:Authorization;
#    acl:accessTo <some-other-file.txt>;
#    acl:agentGroup <group-listing-error.ttl#folks>;
#    acl:mode acl:Read, acl:Write.

Then run NODE_TLS_REJECT_UNAUTHORIZED=0 mocha -g Group

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

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

はじめの一歩

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

調査の方向性

/test/resources/accounts-acl/tim.localhost/group/test-folder/.acl を調べることから始め、示されている :errors ブロックのコメントを解除します。NODE_TLS_REJECT_UNAUTHORIZED=0 mocha -g Group を実行して失敗を再現します。ACL エラーが参照されたグループだけに影響し、すべてのリクエストが HTTP 500 を返すことにならなければ完了です。

索引モデルが issue の本文から書いたものです。

評価

技術スタック
javascript, node.js
領域
authorization, backend
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
明確に書かれている
初心者へのやさしさ
48/100

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

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