nodeSolidServer / nodeSolidServer/node-solid-server
Comply with spec for permissions needed to delete a resource
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
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- 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