Merge pull request #10705 from rainzm/rbac/GetModelProperty

fix(cloudcommon): use rbac authentication dedicated to the list interface in GetModelProperty
This commit is contained in:
Zexi Li
2021-04-13 16:55:52 +08:00
committed by GitHub
+1 -5
View File
@@ -932,11 +932,7 @@ func (dispatcher *DBModelDispatcher) tryGetModelProperty(ctx context.Context, pr
}
if consts.IsRbacEnabled() {
ownerId, err := fetchOwnerId(ctx, dispatcher.modelManager, userCred, query)
if err != nil {
return nil, httperrors.NewGeneralError(err)
}
err = isClassRbacAllowed(dispatcher.modelManager, userCred, ownerId, policy.PolicyActionList, property)
_, _, err := FetchCheckQueryOwnerScope(ctx, userCred, query, dispatcher.modelManager, policy.PolicyActionList, true)
if err != nil {
return nil, err
}