Skip to content

Stellar lookup table#80

Closed
dkgoutham wants to merge 2 commits into
stellarfrom
stellar-lookup-table
Closed

Stellar lookup table#80
dkgoutham wants to merge 2 commits into
stellarfrom
stellar-lookup-table

Conversation

@dkgoutham

Copy link
Copy Markdown

Corrects the Stellar core contract's handling of the address lookup scheme.

PR #76 added a hash → Address storage registry (record_address /
get_address_from_hash + an AddressTable storage key) to the core contract.
The core contract has no from/to fields and never reconstructs an address
from the wire, so that registry was orphaned — wired to nothing. This removes it
and keeps only the piece the core actually owns: the canonical hash_address
primitive.

The storage registry belongs in the NTT manager (the only place that resolves a
32-byte wire value back into a real Address — the inbound recipient). It is
implemented there in a separate NTT PR. hash_address stays a pure function in
the shared wormhole-soroban-client crate so the core contract, NTT, and the
off-chain SDK all compute the identical 32-byte identity.

@dkgoutham
dkgoutham force-pushed the stellar-lookup-table branch from f49451b to e2158f0 Compare June 18, 2026 03:49
@dkgoutham
dkgoutham requested review from Oghma and cupicmarko June 18, 2026 04:50
@dkgoutham dkgoutham closed this Jun 18, 2026
@dkgoutham
dkgoutham deleted the stellar-lookup-table branch June 18, 2026 10:18
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.

1 participant