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

Update to API Go v1.28.1 #660

Merged
merged 6 commits into from
Jul 5, 2021

Conversation

cthulhu-rider
Copy link
Contributor

@cthulhu-rider cthulhu-rider commented Jul 2, 2021

Closes #655.

Leonard Lyubich added 6 commits July 2, 2021 09:02
…equacy

Create `version` package and implement `IsValid` function which checks if
version is not earlier than the genesis one.

Signed-off-by: Leonard Lyubich <[email protected]>
Extended ACL input can have version later than CLI one, and it should not be
downgraded. But the version should be still adequate.

Set `pkg.SDKVersion` on false return of `version.IsValid`.

Signed-off-by: Leonard Lyubich <[email protected]>
…alNodeInfo

Clients can have version later than the server. The server should not deny
these clients.

Signed-off-by: Leonard Lyubich <[email protected]>
Container can have version later than the current lib version, and it should
not be considered as invalid. But it should still have adequate version.

Signed-off-by: Leonard Lyubich <[email protected]>
Storage node should only reflect container Put requests to sidechain w/o any
validation. All verification steps are performed by IR.

Signed-off-by: Leonard Lyubich <[email protected]>
Signed-off-by: Leonard Lyubich <[email protected]>
@cthulhu-rider cthulhu-rider marked this pull request as ready for review July 2, 2021 06:03
@codecov
Copy link

codecov bot commented Jul 2, 2021

Codecov Report

Merging #660 (6cbcf82) into master (a2b622d) will increase coverage by 0.04%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #660      +/-   ##
==========================================
+ Coverage   43.05%   43.09%   +0.04%     
==========================================
  Files         204      205       +1     
  Lines        8072     8078       +6     
==========================================
+ Hits         3475     3481       +6     
  Misses       4256     4256              
  Partials      341      341              
Impacted Files Coverage Δ
pkg/core/container/fmt.go 100.00% <100.00%> (ø)
pkg/core/version/version.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a2b622d...6cbcf82. Read the comment docs.

@alexvanin alexvanin merged commit a2bb59a into nspcc-dev:master Jul 5, 2021
@cthulhu-rider cthulhu-rider deleted the upd-apigo-v1.28.1 branch September 30, 2022 13:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"Unsupported version" error using neofs-cli v0.21.1 with recent dev-env
3 participants