Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

client: Send request meta header once on object PUT stream init #662

Open
cthulhu-rider opened this issue Dec 10, 2024 · 0 comments
Open
Labels
client Issue related to the client enhancement Improving existing functionality I4 No visible changes S3 Minimally significant U4 Nothing urgent

Comments

@cthulhu-rider
Copy link
Contributor

Is your feature request related to a problem? Please describe.

currently, Client attaches https://github.com/nspcc-dev/neofs-api/blob/e66b25d4bf2afc4472023e1e2c2467f694f5a0e1/session/types.proto#L162 to each ObjectService.Put stream message. Although this is not prohibited by the protocol, for chunks messages going after the heading one, this makes no sense. After header is accepted, storage node just reads incoming bytes and ignores meta fields

Describe the solution you'd like

set meta header in the 1st message only

Describe alternatives you've considered

keep and bear corresponding costs

Additional context

Client testing

@cthulhu-rider cthulhu-rider added enhancement Improving existing functionality client Issue related to the client labels Dec 10, 2024
@roman-khimov roman-khimov added U4 Nothing urgent S3 Minimally significant I4 No visible changes labels Dec 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Issue related to the client enhancement Improving existing functionality I4 No visible changes S3 Minimally significant U4 Nothing urgent
Projects
None yet
Development

No branches or pull requests

2 participants