Skip to content

security improvements#1914

Merged
marcello33 merged 3 commits intodevelopfrom
mardizzone/sec
Dec 2, 2025
Merged

security improvements#1914
marcello33 merged 3 commits intodevelopfrom
mardizzone/sec

Conversation

@marcello33
Copy link
Copy Markdown
Collaborator

@marcello33 marcello33 commented Nov 28, 2025

Description

In this PR we implement the following features:

  • Reject difficulty greater than 64 bits
  • Improve security for heimdall gRPC connection
  • go version and dependencies bump
  • standardize lint commands between bor and heimdall

Changes

  • Bugfix (non-breaking change that solves an issue)
  • Hotfix (change that solves an urgent issue, and requires immediate attention)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (change that is not backwards-compatible and/or changes current functionality)
  • Changes only for a subset of nodes

Checklist

  • I have added at least 2 reviewer or the whole pos-v1 team
  • I have added sufficient documentation in code
  • I will be resolving comments - if any - by pushing each fix in a separate commit and linking the commit hash in the comment reply
  • Created a task in Jira and informed the team for implementation in Erigon client (if applicable)
  • Includes RPC methods changes, and the Notion documentation has been updated

Testing

  • I have added unit tests
  • I have added tests to CI
  • I have tested this code manually on local environment
  • I have tested this code manually on remote devnet using express-cli
  • I have tested this code manually on amoy
  • I have created new e2e tests into express-cli

@sonarqubecloud
Copy link
Copy Markdown

@marcello33 marcello33 requested a review from a team November 28, 2025 14:05
@marcello33 marcello33 merged commit 647b061 into develop Dec 2, 2025
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants