SolidOS / SolidOS/solid-ui

acl:Authorization missing for default

Abierto
#172 1 comentario 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

bug enhancement
Lenguaje dominante
TypeScript
Estrellas
153
Forks
46
Merge medio
1 d 8 h
PR fusionados (30 d)
7

Descripción

@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

Guía de contribución

No hay ninguna guía de contribución indexada para este repositorio

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Línea de trabajo

Empieza en src/acl.js, alrededor del código de carga de las reglas predeterminadas en la línea 55, y compáralo después con el tratamiento de acl:Authorization alrededor de la línea 249. Reproduce el escenario /private/test/ en NSS e inspecciona el .acl generado; se considera terminado cuando las reglas predeterminadas heredadas incluyen acl:Authorization.

Escrito por el modelo de indexación a partir del texto del issue.

Evaluación

Stack tecnológico
typescript
Área
authorization
Tipo de issue
Error
Dificultad
2/5
Tiempo estimado
1-3 horas
Estado de actividad
Estancado
Claridad
Bien especificado
Aptitud para principiantes
45/100

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.