nodeSolidServer / nodeSolidServer/node-solid-server

Comply with spec for permissions needed to delete a resource

Open
#1,190 0 comments 2 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Dominant language
JavaScript
Stars
1.8k
Forks
308
PR merge metrics
No merged PRs in 30d

Description

Basically, until https://github.com/solid/web-access-control-spec/issues/47 gets discussed with a proper decision process, let's undo what was done in https://github.com/solid/node-solid-server/issues/729.

I'm not so fussed about the actual behaviour, I just want to start the process of documenting what we consider the spec (see also https://github.com/solid/solid-spec/issues/174), and on which points we are aware of differences between NSS and the spec.

Wherever such differences exist, if we think that NSS is right, then we should update the spec.
And if we think the spec is right (like in this case), we should (at least create a github issue to) fix NSS.

PS: If anyone is aware of any other points on which NSS is not spec-compliant, then let's create issues for each one of those. We can decide to willingly leave NSS broken, but at least we should have github issues that document how NSS is broken.

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the linked web-access-control-spec issue 47, NSS issue 729, and solid-spec issue 174 to compare the permission requirements for deleting a resource. There are no files or tests named in the issue; done means documenting the NSS/spec difference and recording the decision, with a follow-up NSS issue if the implementation should change.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
authorization, backend
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
30/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.