Skip to content

dkg: fixing processkey#4261

Merged
obol-bulldozer[bot] merged 2 commits intomainfrom
pinebit/restore-key-fix
Feb 10, 2026
Merged

dkg: fixing processkey#4261
obol-bulldozer[bot] merged 2 commits intomainfrom
pinebit/restore-key-fix

Conversation

@pinebit
Copy link
Collaborator

@pinebit pinebit commented Jan 29, 2026

This fixes a bug in restoreKey routine that affects "remove operators" edit command.

category: bug
ticket: none

Copy link
Collaborator

@KaloyanTanev KaloyanTanev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would it be possible to have a test case to catch this scenario?

@pinebit
Copy link
Collaborator Author

pinebit commented Jan 29, 2026

Would it be possible to have a test case to catch this scenario?

I was thinking about this, but let's wait for the full report and I suspect we may need to rework a lot more, including more of the tests...

@pinebit pinebit force-pushed the pinebit/restore-key-fix branch from 715e7ec to d9d0b24 Compare January 29, 2026 13:23
@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

❌ Patch coverage is 72.72727% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 56.59%. Comparing base (e2be039) to head (d4f4ae9).
⚠️ Report is 7 commits behind head on main.

Files with missing lines Patch % Lines
dkg/pedersen/reshare.go 0.00% 2 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4261      +/-   ##
==========================================
+ Coverage   56.54%   56.59%   +0.04%     
==========================================
  Files         237      237              
  Lines       31471    31511      +40     
==========================================
+ Hits        17796    17833      +37     
- Misses      11397    11400       +3     
  Partials     2278     2278              

☔ 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.

@pinebit pinebit added the do not merge Indicate to bulldozer bot that this PR should not be merged label Feb 9, 2026
@pinebit pinebit force-pushed the pinebit/restore-key-fix branch from a1dbb17 to d4f4ae9 Compare February 10, 2026 13:19
@sonarqubecloud
Copy link

@pinebit pinebit added merge when ready Indicates bulldozer bot may merge when all checks pass and removed do not merge Indicate to bulldozer bot that this PR should not be merged labels Feb 10, 2026
@obol-bulldozer obol-bulldozer bot merged commit cad6425 into main Feb 10, 2026
11 checks passed
@obol-bulldozer obol-bulldozer bot deleted the pinebit/restore-key-fix branch February 10, 2026 13:41
KaloyanTanev pushed a commit that referenced this pull request Feb 10, 2026
This fixes a bug in `restoreKey` routine that affects "remove operators" edit command.

category: bug
ticket: none
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

3 participants