nodeSolidServer / nodeSolidServer/node-solid-server
:error in .acl causes all requests to error
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- JavaScript
- Sterne
- 1.8k
- Forks
- 308
- PR-Merge-Kennzahlen
- Keine gemergten PRs in 30 T.
Beschreibung
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
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginne mit der Untersuchung von /test/resources/accounts-acl/tim.localhost/group/test-folder/.acl und entferne die Auskommentierung des gezeigten :errors-Blocks. Führe NODE_TLS_REJECT_UNAUTHORIZED=0 mocha -g Group aus, um den Fehler zu reproduzieren. Als abgeschlossen gilt die Änderung, wenn der ACL-Fehler nur die referenzierte Gruppe betrifft und nicht dazu führt, dass jede Anfrage HTTP 500 zurückgibt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- javascript, node.js
- Bereich
- authorization, backend
- Issue-Typ
- Bug
- Schwierigkeit
- 3/5
- Geschätzter Aufwand
- 1-2 Tage
- Aktivitätsstatus
- Veraltet
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 48/100