Skip to content

Conversation

@LeoPatOZ
Copy link
Collaborator

@LeoPatOZ LeoPatOZ commented Nov 5, 2025

Resolves #50

tldr:

  • Adds validation to max_block_range (asserts > 0) on all scanners
  • Asserts count in latest scanner > 0
  • Validation to to / from block in historic mode. Neither can be bigger that latest block
  • In order to achieve this more neatly we expose an shared internal build method. This way the api remains untouched and connect can be used for validation when needed.

@LeoPatOZ LeoPatOZ marked this pull request as ready for review November 11, 2025 13:04
Copy link
Collaborator

@0xNeshi 0xNeshi left a comment

Choose a reason for hiding this comment

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

Close to merging!

0xNeshi
0xNeshi previously approved these changes Nov 13, 2025
Copy link
Collaborator

@0xNeshi 0xNeshi left a comment

Choose a reason for hiding this comment

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

Good job!

@LeoPatOZ LeoPatOZ merged commit eb8ae9a into main Nov 13, 2025
13 checks passed
@LeoPatOZ LeoPatOZ deleted the input-validation branch November 13, 2025 10:15
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.

Add Input Data Validation

3 participants