Merge pull request #2257 from wanjunlei/auditing-authority
add resourcescope to AttributesRecord when determine whether the user…
This commit is contained in:
@@ -707,15 +707,14 @@ func (t *tenantOperator) Auditing(user user.Info, queryParam *auditingv1alpha1.Q
|
|||||||
// those events with empty `ObjectRef.Namespace` will also be listed when user can list all namespaces
|
// those events with empty `ObjectRef.Namespace` will also be listed when user can list all namespaces
|
||||||
if len(queryParam.WorkspaceFilter) == 0 && len(queryParam.ObjectRefNamespaceFilter) == 0 &&
|
if len(queryParam.WorkspaceFilter) == 0 && len(queryParam.ObjectRefNamespaceFilter) == 0 &&
|
||||||
len(queryParam.WorkspaceSearch) == 0 && len(queryParam.ObjectRefNamespaceSearch) == 0 {
|
len(queryParam.WorkspaceSearch) == 0 && len(queryParam.ObjectRefNamespaceSearch) == 0 {
|
||||||
listEvts := authorizer.AttributesRecord{
|
listNs := authorizer.AttributesRecord{
|
||||||
User: user,
|
User: user,
|
||||||
Verb: "list",
|
Verb: "list",
|
||||||
APIGroup: "",
|
|
||||||
APIVersion: "v1",
|
|
||||||
Resource: "namespaces",
|
Resource: "namespaces",
|
||||||
ResourceRequest: true,
|
ResourceRequest: true,
|
||||||
|
ResourceScope: request.ClusterScope,
|
||||||
}
|
}
|
||||||
decision, _, err := t.authorizer.Authorize(listEvts)
|
decision, _, err := t.authorizer.Authorize(listNs)
|
||||||
if err != nil {
|
if err != nil {
|
||||||
klog.Error(err)
|
klog.Error(err)
|
||||||
return nil, err
|
return nil, err
|
||||||
|
|||||||
Reference in New Issue
Block a user