SolidOS / SolidOS/solid-ui

acl:Authorization missing for default

Offen
#172 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

bug enhancement
Vorherrschende Sprache
TypeScript
Sterne
153
Forks
46
Ø Merge
1 T. 8 Std.
Gemergte PRs (30 T.)
7

Beschreibung

@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
  1. Go to /private/ in NSS
  2. Create /private/test/ folder
  3. Set specific sharing for this folder
  4. 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

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Beginne in src/acl.js ungefähr beim Code zum Laden der Standardregeln in Zeile 55 und vergleiche ihn dann mit der Behandlung von acl:Authorization um Zeile 249. Reproduziere das Szenario /private/test/ in NSS und untersuche die erzeugte .acl; abgeschlossen ist die Aufgabe, wenn geerbte Standardregeln acl:Authorization enthalten.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
typescript
Bereich
authorization
Issue-Typ
Bug
Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Aktivitätsstatus
Veraltet
Klarheit
Klar beschrieben
Anfängerfreundlichkeit
45/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.