TrueCloudLab/frostfs-node
20
1
Fork 27
Code Issues 101 Pull requests 5 Projects Releases 1 Activity Actions

[#1319] treeSvc: Do not wrap error from APE
All checks were successful
Vulncheck / Vulncheck (pull_request) Successful in 3m40s
Details
DCO action / DCO (pull_request) Successful in 3m35s
Details
Build / Build Components (1.21) (pull_request) Successful in 4m19s
Details
Build / Build Components (1.22) (pull_request) Successful in 4m17s
Details
Pre-commit hooks / Pre-commit (pull_request) Successful in 4m16s
Details
Tests and linters / Tests (1.21) (pull_request) Successful in 4m16s
Details
Tests and linters / Tests (1.22) (pull_request) Successful in 4m18s
Details
Tests and linters / Staticcheck (pull_request) Successful in 4m34s
Details
Tests and linters / Tests with -race (pull_request) Successful in 4m38s
Details
Tests and linters / Lint (pull_request) Successful in 4m52s
Details
Tests and linters / gopls check (pull_request) Successful in 5m11s
Details

Browse source 
Signed-off-by: Dmitrii Stepanov <d.stepanov@yadro.com>
This commit is contained in:
Dmitrii Stepanov 2024-08-19 15:50:35 +03:00
parent f82b7e1ae3
commit 149f8f4b08
1 changed files with 2 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
pkg/services/tree/ape.go
View file

@ -138,7 +138,7 @@ func (s *Service) checkAPE(ctx context.Context, bt *bearer.Token,
request, err := s.newAPERequest(ctx, namespace, cid, operation, role, publicKey) request, err := s.newAPERequest(ctx, namespace, cid, operation, role, publicKey)
if err != nil { if err != nil {
return apeErr(err) return fmt.Errorf("failed to create ape request: %w", err)
} }
var cr engine.ChainRouter var cr engine.ChainRouter
@ -167,7 +167,7 @@ func (s *Service) checkAPE(ctx context.Context, bt *bearer.Token,
rt := engine.NewRequestTargetExtended(namespace, cid.EncodeToString(), fmt.Sprintf("%s:%s", namespace, publicKey.Address()), groups) rt := engine.NewRequestTargetExtended(namespace, cid.EncodeToString(), fmt.Sprintf("%s:%s", namespace, publicKey.Address()), groups)
status, found, err := cr.IsAllowed(apechain.Ingress, rt, request) status, found, err := cr.IsAllowed(apechain.Ingress, rt, request)
if err != nil { if err != nil {
return apeErr(err) return err
} }
if found && status == apechain.Allow { if found && status == apechain.Allow {
return nil return nil