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.

Sign up for GitHub

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

Jump to bottom

[Improvement] Privilege should have deny mechanism #3342

Closed
qqqttt123 opened this issue May 11, 2024 · 0 comments · Fixed by #3389
Closed

[Improvement] Privilege should have deny mechanism #3342

qqqttt123 opened this issue May 11, 2024 · 0 comments · Fixed by #3389
Assignees
@qqqttt123
Labels
0.5.1 Release v0.5.1 0.6.0 Release v0.6.0 improvement Improvements on everything

Comments

@qqqttt123
Copy link
Contributor

What would you like to be improved?

Privilege should have deny

How should we improve?

Privilege should have deny
@qqqttt123 qqqttt123 added the improvement Improvements on everything label May 11, 2024
@qqqttt123 qqqttt123 changed the title [Improvement] Privilege should have deny [Improvement] Privilege should have deny mechanism May 11, 2024
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 14, 2024
[datastrato#3342] Refactor the role and privilege model
537f481
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
8163833
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
f1a5741
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
a27af57
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
e6f2848
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
4621352
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
84f1ceb
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
612ee96
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
8828bf6
jerqi pushed a commit to qqqttt123/gravitino that referenced this issue May 15, 2024
[datastrato#3342] Refactor the role and privilege model
37bc98d
@qqqttt123 qqqttt123 mentioned this issue May 15, 2024
Merged
jerryshao pushed a commit that referenced this issue May 22, 2024
@jerqi
[#3342] feat(core): Refactor the role and privilege model (#3389)
5a4931b 
### What changes were proposed in this pull request?
Add support for ALLOW/DENY effect for privileges.
Move privileges from the role to the securable object.

### Why are the changes needed?

Fix: #3342

### Does this PR introduce _any_ user-facing change?
Yes. 

### How was this patch tested?
UT passed.

---------

Co-authored-by: Heng Qin <qqtt@123.com>
github-actions bot pushed a commit that referenced this issue May 22, 2024
@jerqi @web-flow
[#3342] feat(core): Refactor the role and privilege model (#3389)
b894e7a 
### What changes were proposed in this pull request?
Add support for ALLOW/DENY effect for privileges.
Move privileges from the role to the securable object.

### Why are the changes needed?

Fix: #3342

### Does this PR introduce _any_ user-facing change?
Yes. 

### How was this patch tested?
UT passed.

---------

Co-authored-by: Heng Qin <qqtt@123.com>
@jerryshao jerryshao mentioned this issue May 22, 2024
Merged
@jerryshao jerryshao added 0.5.1 Release v0.5.1 0.6.0 Release v0.6.0 labels May 22, 2024
qqqttt123 pushed a commit that referenced this issue May 22, 2024
@jerryshao @jerqi
[#3342] feat(core): Refactor the role and privilege model (#3494)
121f2fe 
### What changes were proposed in this pull request?
Add support for ALLOW/DENY effect for privileges.
Move privileges from the role to the securable object.

### Why are the changes needed?

Fix: #3342

### Does this PR introduce _any_ user-facing change?
Yes. 

### How was this patch tested?
UT passed.

Co-authored-by: roryqi <roryqi@apache.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@qqqttt123 qqqttt123

Labels
0.5.1 Release v0.5.1 0.6.0 Release v0.6.0 improvement Improvements on everything
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#3342] feat(core): Refactor the role and privilege model qqqttt123/gravitino
2 participants
@jerryshao @qqqttt123