search

nspcc-dev / neofs-sdk-go

Go implementation of NeoFS SDK
Apache License 2.0
6 stars 14 forks source link

client: Some `ContainerService.GetResponse` fields are ignored #586

Open cthulhu-rider opened 5 months ago

cthulhu-rider commented 5 months ago

NeoFS API client ignores https://github.com/nspcc-dev/neofs-api/blob/62e4ae0f8d62987cb54ab729b16b4f9ed4c0ad4c/container/service.proto#L215-L219

this is either:

  1. bug, and client must start to check these fields
  2. redundant information that should be eliminated from the protocol

i'd stand for 2 since fields are not returned, currently nobody is interested in verification info and is satisfied with the container presence

cthulhu-rider commented 5 months ago

same for GetExtendedACL

roman-khimov commented 5 months ago

If it ended up in the contract, it can be considered valid, no need to check it additionally. But it's not trivial to remove now at the same time.