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] Add the check of privileges #5054

Closed
jerqi opened this issue Sep 30, 2024 · 0 comments · Fixed by #5053
Closed

[Improvement] Add the check of privileges #5054

jerqi opened this issue Sep 30, 2024 · 0 comments · Fixed by #5053
Assignees
@jerqi
Labels
0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 improvement Improvements on everything

Comments

@jerqi
Copy link
Collaborator

jerqi commented Sep 30, 2024

What would you like to be improved?

Add the check of privileges

How should we improve?

Add the check of privileges
@jerqi jerqi added the improvement Improvements on everything label Sep 30, 2024
@jerqi jerqi mentioned this issue Sep 30, 2024
Merged
@jerqi jerqi self-assigned this Sep 30, 2024
@jerqi jerqi added 0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 labels Sep 30, 2024
github-actions bot pushed a commit that referenced this issue Sep 30, 2024
@jerqi @jerryshao @web-flow
[#5054] improvement(api,server): Add the check of privileges (#5053)
7e4738b 
### What changes were proposed in this pull request?

Add the check of privileges

### Why are the changes needed?

Fix: #5054

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Add some UTs

---------

Co-authored-by: Jerry Shao <jshao@apache.org>
@github-actions github-actions bot mentioned this issue Sep 30, 2024
Merged
yuqi1129 pushed a commit that referenced this issue Sep 30, 2024
@github-actions @jerqi @jerryshao
[#5054] improvement(api,server): Add the check of privileges (#5056)
bfdbf31 
### What changes were proposed in this pull request?

Add the check of privileges

### Why are the changes needed?

Fix: #5054

### Does this PR introduce _any_ user-facing change?
No

### How was this patch tested?
Add some UTs

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

@jerqi jerqi

Labels
0.6.1 Release v0.6.1 0.7.0 Release v0.7.0 improvement Improvements on everything
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[#5054] improvement(api,server): Add the check of privileges qqqttt123/gravitino
1 participant
@jerqi