Skip to content

Use zcash_primitives to perform parsing for protocol types.#4

Merged
wolfmcnally merged 7 commits intoBlockchainCommons:masterfrom
nuttycom:primitives_tx_parsing
May 12, 2025
Merged

Use zcash_primitives to perform parsing for protocol types.#4
wolfmcnally merged 7 commits intoBlockchainCommons:masterfrom
nuttycom:primitives_tx_parsing

Conversation

@nuttycom
Copy link
Contributor

@nuttycom nuttycom commented May 3, 2025

This replaces locally-defined parsing logic with the same versions of zcash_primitives and bridgetree as zcashd uses. By doing so, we can ensure compatibility and gain access to the protocol data that is needed for type-safe conversion into the zewif formats.

@nuttycom nuttycom force-pushed the primitives_tx_parsing branch from 06d1fe1 to 67117c3 Compare May 3, 2025 03:58
@nuttycom nuttycom force-pushed the primitives_tx_parsing branch 7 times, most recently from e7c76b5 to c6babb2 Compare May 10, 2025 03:21
nuttycom added 5 commits May 9, 2025 21:26
This replaces locally-defined parsing logic with `zcashd`'s parsing
code. By using the same versions of `zcash_primitives` and `bridgetree`
as `zcashd` uses, we can ensure compatibility and gain access to the
protocol data that is needed for type-safe conversion into the `zewif`
formats.

This also moves the `parser` infrastructure here from the `zewif` crate.
The encoding handled by these parsers is `zcashd`-specific, and should
not be used by the wider ecosystem.
This also partitions functionality by subprotocol relevance, adding the
`transparent`, `sprout`, `sapling`, and `orchard` submodules.
@nuttycom nuttycom force-pushed the primitives_tx_parsing branch from c6babb2 to 01a914e Compare May 10, 2025 03:28
@nuttycom nuttycom marked this pull request as ready for review May 10, 2025 03:31
Copy link
Contributor

@wolfmcnally wolfmcnally left a comment

Choose a reason for hiding this comment

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

LGTM

@wolfmcnally wolfmcnally merged commit ad8b8d5 into BlockchainCommons:master May 12, 2025
7 of 8 checks passed
@nuttycom nuttycom deleted the primitives_tx_parsing branch May 12, 2025 15:36
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