Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Make DAC write permission more granular #3218

Draft
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

shahzadlone
Copy link
Member

@shahzadlone shahzadlone commented Nov 7, 2024

Relevant issue(s)

Resolves #2905

Description

Split write into update and delete permission.

For reviewers

[ ] Reviewing the draft please let me know if anything in the splitting logic seems odd to others, we will now not have any write perm, only separate update and delete permission.

How has this been tested?

  • todo: Update previous test once migration of policyID is done
  • todo: Split previous write tests

@shahzadlone shahzadlone added feature New feature or request area/acp Related to the acp (access control) system labels Nov 7, 2024
@shahzadlone shahzadlone self-assigned this Nov 7, 2024
@shahzadlone shahzadlone force-pushed the lone/feat/granular-dpi-mutation-perms branch from e66b85c to 4c5abd5 Compare January 2, 2025 20:06
@shahzadlone shahzadlone force-pushed the lone/feat/granular-dpi-mutation-perms branch from 4c5abd5 to 19a52d8 Compare January 2, 2025 20:06
@shahzadlone shahzadlone requested a review from a team January 2, 2025 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/acp Related to the acp (access control) system feature New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Split ACP write perm into delete and update
1 participant