Skip to content

[fix][relayer][core] preserve blockchain properties on partial update - #78

Merged
zouxyan merged 1 commit into
AntChainOpenLabs:mainfrom
fengjy73:fix/relayer-partial-blockchain-update
Aug 7, 2026
Merged

[fix][relayer][core] preserve blockchain properties on partial update#78
zouxyan merged 1 commit into
AntChainOpenLabs:mainfrom
fengjy73:fix/relayer-partial-blockchain-update

Conversation

@fengjy73

@fengjy73 fengjy73 commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

CLI/API updates may provide only a subset of blockchain properties, such as heterogeneous_bbc_context. The previous flow merged the partial config into the existing BlockchainMeta, but then persisted a newly constructed meta from only the partial properties. That could wipe fields that were not present in the update payload.

Affected fields include plugin server metadata, init block height, AM/SDP/PTC contract addresses, and domain registration state.

  • Persist the merged BlockchainMeta in BlockchainManager.updateBlockchain(...).
  • Merge ptc_contract_address in BlockchainMeta.updateProperties(...).

@zouxyan zouxyan left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Reviewed commit 132ad53. Persisting the merged BlockchainMeta correctly preserves fields omitted from partial updates, and the added PTC contract address merge is consistent with the existing AM/SDP behavior. No blocking correctness issues found.

@zouxyan
zouxyan merged commit 43fda79 into AntChainOpenLabs:main Aug 7, 2026
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.

2 participants