TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 108 Pull requests 9 Projects Releases 2 Activity Actions

[#869] object: Pass just CID to chain router
All checks were successful
DCO action / DCO (pull_request) Successful in 11m55s
Details
Vulncheck / Vulncheck (pull_request) Successful in 13m1s
Details
Build / Build Components (1.20) (pull_request) Successful in 13m19s
Details
Build / Build Components (1.21) (pull_request) Successful in 13m15s
Details
Tests and linters / Staticcheck (pull_request) Successful in 3m31s
Details
Tests and linters / Lint (pull_request) Successful in 5m39s
Details
Tests and linters / Tests (1.20) (pull_request) Successful in 7m55s
Details
Tests and linters / Tests (1.21) (pull_request) Successful in 8m6s
Details
Tests and linters / Tests with -race (pull_request) Successful in 8m44s
Details

Browse source 
* Do not convert CID from request to native-schema resource
  format - this step is unneccessary for APE.

Signed-off-by: Airat Arifullin <a.arifullin@yadro.com>
This commit is contained in:
Airat Arifullin 2023-12-13 15:29:17 +03:00
parent 416693847c
commit ef0551c347
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
pkg/services/object/acl/ape.go
View file

@ -26,7 +26,7 @@ func (c *apeCheckerImpl) CheckIfRequestPermitted(reqInfo v2.RequestInfo) error {
request := new(Request)
request.FromRequestInfo(reqInfo)
cnrTarget := getResource(reqInfo).Name()
cnrTarget := reqInfo.ContainerID().EncodeToString()
status, ruleFound, err := c.chainRouter.IsAllowed(apechain.Ingress, policyengine.NewRequestTargetWithContainer(cnrTarget), request)
if err != nil {