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

feat: add user perms & view decorator response_perms #1599

Merged
merged 5 commits into from
Oct 20, 2021
Merged

feat: add user perms & view decorator response_perms #1599

merged 5 commits into from
Oct 20, 2021

Conversation

jamesgetx
Copy link
Collaborator

变更点(Changes)

  • 提供计算命名空间的全局唯一ID，用于注册到权限中心
  • 添加view装饰器response_perms，用于注入权限数据到web_annotations中
  • 添加查询用于perms接口

@jamesgetx jamesgetx linked an issue Oct 14, 2021 that may be closed by this pull request
权限框架分pr合入master #1537
Closed
2 tasks
narasux
narasux reviewed Oct 18, 2021
View reviewed changes
bcs-app/backend/iam/open_apis/provider/cluster.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/open_apis/provider/namespace.py Show resolved Hide resolved
bcs-app/backend/iam/open_apis/provider/templateset.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/open_apis/provider/templateset.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/perm_maker.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/permissions/resources/namespace.py Outdated Show resolved Hide resolved
bcs-app/backend/tests/iam/fake_iam.py Outdated Show resolved Hide resolved
bcs-app/backend/tests/iam/fake_iam.py Outdated Show resolved Hide resolved
piglei
piglei reviewed Oct 18, 2021
View reviewed changes
bcs-app/backend/components/paas_cc.py Outdated Show resolved Hide resolved
bcs-app/backend/helm/helm/utils/repo_bk.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/open_apis/provider/cluster.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/open_apis/provider/cluster.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/open_apis/provider/namespace.py Show resolved Hide resolved
bcs-app/backend/iam/permissions/decorators.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/permissions/decorators.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/permissions/resources/namespace.py Outdated Show resolved Hide resolved
bcs-app/backend/iam/permissions/resources/namespace.py Outdated Show resolved Hide resolved
bcs-app/backend/tests/iam/fake_iam.py Outdated Show resolved Hide resolved
piglei
piglei approved these changes Oct 20, 2021
View reviewed changes
Copy link
Collaborator

@piglei piglei left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dellkeji dellkeji merged commit 12e5a8e into Tencent:master Oct 20, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dellkeji dellkeji dellkeji left review comments

@narasux narasux narasux left review comments

@piglei piglei piglei approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

权限框架分pr合入master
4 participants
@jamesgetx @piglei @dellkeji @narasux