Skip to content

Conversation

@KaloyanTanev
Copy link
Collaborator

Add DKG changes in v1.9.0-rc2

category: misc
ticket: none

KaloyanTanev and others added 10 commits February 10, 2026 16:59
This would help us to easily manoeuvre through the list of actions in the UI.

category: refactor
ticket: none
Improved nodes counts validation for reshare DKG.

category: refactor
ticket: none
Added threshold validation for `RunDKG` and `RunReshareDKG`. Previous behavior delegated the check down the stack to allow underlying library (kyber) to validate the threshold. Given that Kyber has weak validation that can lead to weird late errors, it is more convenient to verify the threshold upfront.

Additionally added verification such that users cannot remove all nodes with "remove operators" command.

category: refactor
ticket: none
Added dkg p2p wire message size validation.

category: refactor
ticket: none
Verify that the actual sender's peer ID matches the claimed peer index in nodesigs.

category: refactor
ticket: none
Using `io.LimitReader` in `FetchDefinition`, the maximum allowed definition size is 16Mb.

category: refactor
ticket: none
Improving frostp2p code quality.

category: refactor
ticket: none
This fixes a bug in `restoreKey` routine that affects "remove operators" edit command.

category: bug
ticket: none
Improves `reshare.go` checks:

1. ` restoreCommits` out-of-bounds checks for `shareNum`
2.  polynomial commitment validation during reshare (against expected validator pubkey)

category: refactor
ticket: none
Better nonce generation and variation for dkg.

category: refactor
ticket: none
@sonarqubecloud
Copy link

@KaloyanTanev KaloyanTanev added the merge when ready Indicates bulldozer bot may merge when all checks pass label Feb 10, 2026
@obol-bulldozer obol-bulldozer bot merged commit 83793cb into main-v1.9 Feb 10, 2026
11 checks passed
@obol-bulldozer obol-bulldozer bot deleted the kalo/v1.9.0-rc2 branch February 10, 2026 15:10
@codecov
Copy link

codecov bot commented Feb 10, 2026

Codecov Report

❌ Patch coverage is 55.71429% with 62 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.59%. Comparing base (fa6ffbf) to head (05ddf77).
⚠️ Report is 1 commits behind head on main-v1.9.

Files with missing lines Patch % Lines
dkg/pedersen/reshare.go 49.36% 31 Missing and 9 partials ⚠️
dkg/frostp2p.go 42.10% 7 Missing and 4 partials ⚠️
dkg/pedersen/dkg.go 57.89% 4 Missing and 4 partials ⚠️
dkg/protocolsteps.go 71.42% 1 Missing and 1 partial ⚠️
dkg/sync/server.go 66.66% 1 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##           main-v1.9    #4313      +/-   ##
=============================================
+ Coverage      56.56%   56.59%   +0.03%     
=============================================
  Files            237      237              
  Lines          31471    31559      +88     
=============================================
+ Hits           17800    17861      +61     
- Misses         11390    11412      +22     
- Partials        2281     2286       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

branch-release merge when ready Indicates bulldozer bot may merge when all checks pass

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants