Skip to content

Adapt Chain Identifier to return full 32 byte digest#25186

Merged
tpham-mysten merged 1 commit intomainfrom
tpham-chain-identifier-base58
Jan 30, 2026
Merged

Adapt Chain Identifier to return full 32 byte digest#25186
tpham-mysten merged 1 commit intomainfrom
tpham-chain-identifier-base58

Conversation

@tpham-mysten
Copy link
Copy Markdown
Contributor

Description

The 4-byte hex representation is a holdover from JSON-RPC. We should return the full Base58 encoded genesis digest instead, as it is used in other places (e.g. TransactionExpiration) with that format.

Test plan

How did you test the new or updated feature?

cargo nextest run -p sui-indexer-alt-graphql

Release notes

Check each box that your changes affect. If none of the boxes relate to your changes, release notes aren't required.

For each box you select, include information after the relevant heading that describes the impact of your changes that a user might notice and any actions they must take to implement updates.

  • Protocol:
  • Nodes (Validators and Full nodes):
  • gRPC:
  • JSON-RPC:
  • GraphQL: chainIdentifier query now returns full Base58-encoded 32 byte digest
  • CLI:
  • Rust SDK:
  • Indexing Framework:

@vercel
Copy link
Copy Markdown

vercel bot commented Jan 30, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
sui-docs Ready Ready Preview, Comment Jan 30, 2026 3:15am
2 Skipped Deployments
Project Deployment Actions Updated (UTC)
multisig-toolkit Ignored Ignored Preview Jan 30, 2026 3:15am
sui-kiosk Ignored Ignored Preview Jan 30, 2026 3:15am

Request Review

@tpham-mysten tpham-mysten merged commit 696fdee into main Jan 30, 2026
73 of 74 checks passed
@tpham-mysten tpham-mysten deleted the tpham-chain-identifier-base58 branch January 30, 2026 17:30
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