acl:Authorization missing for default
Nessuno ha ancora preso questa issue.
- Lingua principale
- TypeScript
- Stelle
- 153
- Fork
- 46
- Merge medio
- 1g 8h
- PR unite (30g)
- 7
Descrizione
@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
Guida per i contributori
Nessuna guida per i contributori indicizzata per questo repository
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Direzione di ricerca
Inizia in src/acl.js, intorno al codice di caricamento delle regole predefinite alla riga 55, quindi confrontalo con la gestione di acl:Authorization intorno alla riga 249. Riproduci lo scenario /private/test/ in NSS e ispeziona il file .acl generato; il lavoro è completo quando le regole predefinite ereditate includono acl:Authorization.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Valutazione
- Stack tecnologico
- typescript
- Ambito
- authorization
- Tipo di issue
- Bug
- Difficoltà
- 2/5
- Tempo stimato
- 1-3 ore
- Stato di attività
- Ferma
- Chiarezza
- Specificata chiaramente
- Idoneità per principianti
- 45/100