acl:Authorization missing for default
まだ誰も着手していません。
- 主要言語
- TypeScript
- スター
- 153
- フォーク
- 46
- 平均マージ
- 1日 8時間
- マージ済み PR（30日）
- 7
説明
@Bourgeoa found an inconsistency while testing my solid-acl-parser library: When a folder loads the default rules from a parent, no acl:Authorization is added to these rules. As a result my parser didn't notice them.
Steps to reproduce
- Go to /private/ in NSS
- Create /private/test/ folder
- Set specific sharing for this folder
- Inspect /private/test/.acl
@prefix : <#>.
@prefix n0: <http://www.w3.org/ns/auth/acl#>.
@prefix test: <./>.
@prefix c: </profile/card#>.
:owner
n0:accessTo test:;
n0:agent c:me;
n0:default test:;
n0:mode n0:Control, n0:Read, n0:Write.
Possible Fix
In the acl.js code, there is a line where acl:Authorization is added (here), but I afaik the function which loads the default doesn't call it.
I think the acl:Authorization should also be added here (can't test and verify it though):
https://github.com/solid/solid-ui/blob/d47d8a0e875de5ae4649fdd548400113b9f24c8d/src/acl.js#L55
コントリビューションガイド
このリポジトリのコントリビューションガイドは索引されていません
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
src/acl.js の55行目付近にあるデフォルトルールの読み込みコードから始め、249行目付近の acl:Authorization の処理と比較してください。NSS で /private/test/ シナリオを再現し、生成された .acl を調べます。継承されたデフォルトルールに acl:Authorization が含まれていれば完了です。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- typescript
- 領域
- authorization
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 停滞
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 45/100