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.

Sign up for GitHub

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

feat: parallel partial witness handling in the partial witness actor #12656

Merged
merged 13 commits into from
Jan 13, 2025
Merged

feat: parallel partial witness handling in the partial witness actor #12656

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
80a6dff
spawn thread in handle_partial_encoded_state_witness
stedfn Dec 18, 2024
3613619
remove file
stedfn Dec 18, 2024
c12d23b
clippy
stedfn Dec 18, 2024
1de100c
modify handle_partial_encoded_state_witness_forward
stedfn Dec 19, 2024
3f63d03
.
stedfn Dec 20, 2024
c74680e
Merge branch 'master' into stefan/improved_parallel
stedfn Jan 3, 2025
b5102e8
cleanup
stedfn Jan 3, 2025
53246b4
log error if store_partial_encoded_state_witness fails
stedfn Jan 3, 2025
445ff01
address comments
stedfn Jan 7, 2025
0043c54
Merge branch 'master' into stefan/improved_parallel
stedfn Jan 7, 2025
aabeb72
address comment
stedfn Jan 13, 2025
0ea0624
add todo comment to ban peers that send invalid partial witnesses
stedfn Jan 13, 2025
79acde8
Merge branch 'master' into stefan/improved_parallel
stedfn Jan 13, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
add todo comment to ban peers that send invalid partial witnesses
  • Loading branch information
@stedfn
stedfn committed Jan 13, 2025
commit 0ea062415b6e60551facb6913e254e46aa15a952
2 changes: 2 additions & 0 deletions chain/client/src/stateless_validation/partial_witness/partial_witness_actor.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -415,6 +415,7 @@ impl PartialWitnessActor {
));
}
Ok(false) => {
// TODO: ban sending peer
tracing::warn!(
target: "client",
"Received invalid partial encoded state witness"
Expand Down Expand Up @@ -461,6 +462,7 @@ impl PartialWitnessActor {
}
}
Ok(false) => {
// TODO: ban sending peer
tracing::warn!(
stedfn marked this conversation as resolved.
Show resolved Hide resolved
target: "client",
"Received invalid partial encoded state witness"
Expand Down
Loading