BlueBrain / BlueBrain/nexus

When overriding Initial ACLs on / check for the permissions

Open
#352 1 comment 0 reactions 0 assignees View on GitHub
bug delta
Dominant language
Scala
Stars
301
Forks
76
PR merge metrics
No merged PRs in 30d

Description

Since the initial ACLs on `/` are volatile for `Anonymous`, once those are overridden (an append or a replace happens) they are not available anymore. This could lead to a situation where no one has access to do anything in the system if the newly set permissions are not correct.

We should check on the ACLs state machine that when performing a `replace` or `addition` and the previous state is `Initial`, we will reject if the newly passed permissions do not contain at least `acls/read` and `acls/write`

Contributor guide

Open the contributing guide

Research direction

Start with the ACL state machine and trace the replace and addition transitions from the Initial state. Verify how permissions are represented and validated, then add coverage showing that transitions lacking both acls/read and acls/write are rejected while valid permissions are accepted.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
authorization, security
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.