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

rpcsrv: enforce default config values on server creation if malformed value specified #3107

Merged
merged 5 commits into from
Aug 29, 2023

Conversation

AnnaShaleva
Copy link
Member

Ref. #3106. This PR makes the life of NeoFS IR supernode better.

@AnnaShaleva AnnaShaleva added the rpc RPC server and client label Aug 29, 2023
@AnnaShaleva AnnaShaleva added this to the v0.102.0 milestone Aug 29, 2023
@roman-khimov
Copy link
Member

Do we need default values in pkg/config/config.go then?

Do not use RPC configuration constructor for this, some external services
may skip this part.

Signed-off-by: Anna Shaleva <[email protected]>
Do not use RPC configuration constructor for this, some external services
may skip this part.

Signed-off-by: Anna Shaleva <[email protected]>
Do not use RPC configuration constructor for this, some external services
may skip this part.

Signed-off-by: Anna Shaleva <[email protected]>
Do not use RPC configuration constructor for this, some external services
may skip this part.

Signed-off-by: Anna Shaleva <[email protected]>
@AnnaShaleva AnnaShaleva marked this pull request as ready for review August 29, 2023 10:29
@AnnaShaleva
Copy link
Member Author

Tests pass now.

@codecov
Copy link

codecov bot commented Aug 29, 2023

Codecov Report

Merging #3107 (b89078c) into master (b4dff7b) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #3107   +/-   ##
=======================================
  Coverage   84.81%   84.81%           
=======================================
  Files         329      329           
  Lines       44222    44232   +10     
=======================================
+ Hits        37507    37517   +10     
  Misses       5206     5206           
  Partials     1509     1509           
Files Changed Coverage Δ
pkg/config/config.go 85.00% <ø> (-1.96%) ⬇️
pkg/services/rpcsrv/server.go 80.58% <100.00%> (+0.13%) ⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@roman-khimov roman-khimov merged commit 0b67fa9 into master Aug 29, 2023
14 of 18 checks passed
@roman-khimov roman-khimov deleted the default-iter-items-conut branch August 29, 2023 12:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rpc RPC server and client
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants