Skip to content

Security: Anti-Double-Mining Hash Integrity#2878

Closed
MichaelSovereign wants to merge 17 commits intoScottcjn:mainfrom
MichaelSovereign:security-fix/anti-double-mining-hash
Closed

Security: Anti-Double-Mining Hash Integrity#2878
MichaelSovereign wants to merge 17 commits intoScottcjn:mainfrom
MichaelSovereign:security-fix/anti-double-mining-hash

Conversation

@MichaelSovereign
Copy link
Copy Markdown
Contributor

Summary

This PR improves the machine identity hashing mechanism in the module to prevent potential identity collisions.

Changes

  1. Full Hash Utilization: Replaced the 16-character truncated SHA-256 hash with the full 64-character hex string. This significantly reduces the probability of identity collisions where two distinct physical machines might be incorrectly identified as the same machine, leading to denial of rewards.
  2. Identity Robustness: Ensures that the machine identity is globally unique and cryptographically sound.

Closes #7458

@MichaelSovereign MichaelSovereign requested a review from Scottcjn as a code owner May 2, 2026 05:17
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) node Node server related tests Test suite changes size/L PR: 201-500 lines labels May 2, 2026
@Scottcjn Scottcjn closed this May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) node Node server related size/L PR: 201-500 lines tests Test suite changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants