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

[#1480] objsvc: Remove useless stream wrappers
All checks were successful
DCO action / DCO (pull_request) Successful in 1m54s
Details
Vulncheck / Vulncheck (pull_request) Successful in 2m39s
Details
Tests and linters / Run gofumpt (pull_request) Successful in 2m55s
Details
Tests and linters / Staticcheck (pull_request) Successful in 3m32s
Details
Tests and linters / Lint (pull_request) Successful in 4m8s
Details
Pre-commit hooks / Pre-commit (pull_request) Successful in 4m28s
Details
Tests and linters / gopls check (pull_request) Successful in 4m26s
Details
Build / Build Components (pull_request) Successful in 4m42s
Details
Tests and linters / Tests (pull_request) Successful in 6m25s
Details
Tests and linters / Tests with -race (pull_request) Successful in 7m45s
Details

Browse source 
Signed-off-by: Evgenii Stratonikov <e.stratonikov@yadro.com>
This commit is contained in:
Evgenii Stratonikov 2024-11-08 10:10:06 +03:00
parent 05c392b050
commit 1cdded6910
Signed by: fyrchik
SSH key fingerprint: SHA256:m/TTwCzjnRkXgnzEx9X92ccxy1CcVeinOgDb3NPWWmg
1 changed files with 3 additions and 33 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

36
pkg/services/object/acl/v2/service.go
View file

@ -41,18 +41,6 @@ type patchStreamBasicChecker struct {
nonFirstSend bool
}
type getStreamBasicChecker struct {
object.GetObjectStream
}
type rangeStreamBasicChecker struct {
object.GetObjectRangeStream
}
type searchStreamBasicChecker struct {
object.SearchStream
}
// Option represents Service constructor option.
type Option func(*cfg)
@ -212,9 +200,7 @@ func (b Service) Get(request *objectV2.GetRequest, stream object.GetObjectStream
reqInfo.obj = obj
return b.next.Get(request, &getStreamBasicChecker{
GetObjectStream: newWrappedGetObjectStreamStream(stream, reqInfo),
})
return b.next.Get(request, newWrappedGetObjectStreamStream(stream, reqInfo))
}
func (b Service) Put() (object.PutObjectStream, error) {
@ -318,9 +304,7 @@ func (b Service) Search(request *objectV2.SearchRequest, stream object.SearchStr
return err
}
return b.next.Search(request, &searchStreamBasicChecker{
SearchStream: newWrappedSearchStream(stream, reqInfo),
})
return b.next.Search(request, newWrappedSearchStream(stream, reqInfo))
}
func (b Service) Delete(
@ -413,9 +397,7 @@ func (b Service) GetRange(request *objectV2.GetRangeRequest, stream object.GetOb
reqInfo.obj = obj
return b.next.GetRange(request, &rangeStreamBasicChecker{
GetObjectRangeStream: newWrappedRangeStream(stream, reqInfo),
})
return b.next.GetRange(request, newWrappedRangeStream(stream, reqInfo))
}
func requestContext(ctx context.Context, reqInfo RequestInfo) context.Context {
@ -627,18 +609,6 @@ func (p putStreamBasicChecker) CloseAndRecv(ctx context.Context) (*objectV2.PutR
return p.next.CloseAndRecv(ctx)
}
func (g *getStreamBasicChecker) Send(resp *objectV2.GetResponse) error {
return g.GetObjectStream.Send(resp)
}
func (g *rangeStreamBasicChecker) Send(resp *objectV2.GetRangeResponse) error {
return g.GetObjectRangeStream.Send(resp)
}
func (g *searchStreamBasicChecker) Send(resp *objectV2.SearchResponse) error {
return g.SearchStream.Send(resp)
}
func (p *patchStreamBasicChecker) Send(ctx context.Context, request *objectV2.PatchRequest) error {
body := request.GetBody()
if body == nil {