nodeSolidServer / nodeSolidServer/node-solid-server

:error in .acl causes all requests to error

未关闭
#1,263 0 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

还没有人认领这个 Issue。

主要语言
JavaScript
星标
1.8k
派生
308
PR 合并指标
30 天内没有已合并 PR

描述

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

贡献指南

打开贡献指南

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

调研方向

首先检查 /test/resources/accounts-acl/tim.localhost/group/test-folder/.acl,并取消注释其中显示的 :errors 块。运行 NODE_TLS_REJECT_UNAUTHORIZED=0 mocha -g Group 以复现该失败。完成的标准是 ACL 错误只影响被引用的组,而不会导致每个请求都返回 HTTP 500。

由索引模型根据 Issue 内容生成。

评估

技术栈
javascript, node.js
领域
authorization, backend
Issue 类型
缺陷
难度
3/5
预计耗时
1-2 天
活跃度
停滞
描述清晰度
描述清楚
新手友好度
48/100

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。