nodeSolidServer / nodeSolidServer/node-solid-server

:error in .acl causes all requests to error

Aperta
#1,263 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Lingua principale
JavaScript
Stelle
1.8k
Fork
308
Metriche di merge delle PR
Nessuna PR unita negli ultimi 30g

Descrizione

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

Guida per i contributori

Apri la guida per i contributori

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Direzione di ricerca

Inizia ispezionando /test/resources/accounts-acl/tim.localhost/group/test-folder/.acl e rimuovi il commento dal blocco :errors mostrato. Esegui NODE_TLS_REJECT_UNAUTHORIZED=0 mocha -g Group per riprodurre il problema. Il lavoro è completato quando l’errore ACL interessa solo il gruppo referenziato e non fa sì che ogni richiesta restituisca HTTP 500.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Valutazione

Stack tecnologico
javascript, node.js
Ambito
authorization, backend
Tipo di issue
Bug
Difficoltà
3/5
Tempo stimato
1-2 giorni
Stato di attività
Ferma
Chiarezza
Specificata chiaramente
Idoneità per principianti
48/100

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.