Skip to content

Validate query uses capabilities from blockchain config, not candidate block#30

Merged
bvscd merged 14 commits intorelease/node/v0.4.0from
fix_use_config_in_validate
Mar 25, 2026
Merged

Validate query uses capabilities from blockchain config, not candidate block#30
bvscd merged 14 commits intorelease/node/v0.4.0from
fix_use_config_in_validate

Conversation

@yaroslavser
Copy link
Copy Markdown

@yaroslavser yaroslavser commented Mar 24, 2026

Fix for issue #28

Copilot AI review requested due to automatic review settings March 24, 2026 11:16
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates validator transaction checking to rely on the network configuration’s global version/capabilities (from ConfigParams) rather than the optional per-block gen_software() field, and bumps the node crate version.

Changes:

  • Use base.config_params.global_version() and base.config_params.capabilities() when constructing BlockchainConfig for transaction validation.
  • Bump src/node crate version from 0.3.0 to 0.4.0 (and update Cargo.lock accordingly).

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/node/src/validator/validate_query.rs Builds BlockchainConfig for transaction validation using values from config params (authoritative network config).
src/node/Cargo.toml Increments the node crate version to 0.4.0.
src/Cargo.lock Updates lockfile entry for the node package version.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@bvscd bvscd changed the title Validate query uses version and capabilities from config Validate query uses version and capabilities from blockchain config, not candidate block Mar 24, 2026
@bvscd bvscd changed the base branch from master to release/node/v0.4.0 March 24, 2026 11:23
@yaroslavser yaroslavser changed the title Validate query uses version and capabilities from blockchain config, not candidate block Validate query uses capabilities from blockchain config, not candidate block Mar 25, 2026
@bvscd bvscd merged commit 82644c4 into release/node/v0.4.0 Mar 25, 2026
5 checks passed
@bvscd bvscd deleted the fix_use_config_in_validate branch March 25, 2026 13:47
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.

4 participants