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

dbft: do not send PrepareRequest immediately after Recovery message #75

Merged
merged 1 commit into from
May 15, 2023

Conversation

AnnaShaleva
Copy link
Member

We have timer, and we should follow it. Close #74.

@AnnaShaleva AnnaShaleva marked this pull request as draft April 21, 2023 09:21
@AnnaShaleva
Copy link
Member Author

Draft until the neo-project/neo-modules#799 is resolved.

We have timer, and we should follow it. Close #74.

Signed-off-by: Anna Shaleva <anna@nspcc.ru>
@codecov
Copy link

codecov bot commented Apr 21, 2023

Codecov Report

Merging #75 (5a6d1e5) into master (0108387) will increase coverage by 0.11%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
+ Coverage   75.05%   75.17%   +0.11%     
==========================================
  Files          24       24              
  Lines        1311     1309       -2     
==========================================
  Hits          984      984              
+ Misses        272      271       -1     
+ Partials       55       54       -1     
Impacted Files Coverage Δ
dbft.go 68.73% <ø> (+0.36%) ⬆️

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

@AnnaShaleva AnnaShaleva marked this pull request as ready for review May 15, 2023 11:35
@roman-khimov roman-khimov merged commit 25db6ba into master May 15, 2023
@roman-khimov roman-khimov deleted the fix-recovery-resp branch May 15, 2023 11:36
AnnaShaleva added a commit to nspcc-dev/neo-go that referenced this pull request May 16, 2023
Send PrepareRequest on timeout only: nspcc-dev/dbft#75.
AnnaShaleva added a commit to nspcc-dev/neo-go that referenced this pull request May 16, 2023
Send PrepareRequest on timeout only: nspcc-dev/dbft#75.

Signed-off-by: Anna Shaleva <shaleva.ann@nspcc.ru>
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.

RecoveryRequest triggers PrepareRequest from Primary even if the time is not yet came to send it
2 participants