Skip to content

blockchain: gate zero-threshold WeightedMultiSig checks behind Permissionless#982

Open
2dvorak wants to merge 1 commit into
kaiachain:devfrom
2dvorak:fix/permissionless-gate-zero-threshold
Open

blockchain: gate zero-threshold WeightedMultiSig checks behind Permissionless#982
2dvorak wants to merge 1 commit into
kaiachain:devfrom
2dvorak:fix/permissionless-gate-zero-threshold

Conversation

@2dvorak

@2dvorak 2dvorak commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Proposed changes

  • Gate the zero-threshold WeightedMultiSig checks (CheckInstallable/Validate) and the validateSender zero-signature check behind the Permissionless fork, so that an account already installed a Threshold=0 key would not diverge when replaying historical blocks.
  • Update tests to assert both pre-fork (accepted, replayable) and post-fork (rejected) behavior.

Types of changes

  • 🐛 Bug fix
  • ✨ Non-hardfork changes (node upgrade not required)
  • 💥 Hardfork / consensus-breaking changes
  • 🧪 Test improvements
  • 🧰 CI / build tool
  • ♻️ Chore / Refactor / Non-functional changes

Checklist

  • 📖 I have read the CONTRIBUTING GUIDELINES doc
  • 📝 I have signed in the PR comment I have read the CLA Document and I hereby sign the CLA in first time contribute after having read CLA
  • 🟢 Lint and unit tests pass locally with my changes ($ make test)

Related issues

Further comments

…sionless fork

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@2dvorak 2dvorak self-assigned this Jul 16, 2026
@2dvorak
2dvorak marked this pull request as ready for review July 16, 2026 07:50
@hyunsooda hyunsooda added the do not merge Do not merge just yet label Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge Do not merge just yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants