Skip to content

Conversation

@Davidson-Souza
Copy link
Member

Description and Notes

A UtreexoProof struct contains three main items: a Merkle Proof, the position targets and the preimage for each UTXO being spent. RequestSelection allows you to choose which one of the structs do you want independently.

This is serialized as a Bitmap, but the api uses a builder pattern where you can use a setter to tell which ones do you want.

There's also the request_all helper that just sets all Bits, and is useful to avoid having to call all three setters.

How to verify the changes you have done?

Run the latest utreexod main on signet and connect floresta with and without this change. On master it should error out with a tx validation error for missing txout.

@Davidson-Souza Davidson-Souza self-assigned this Feb 5, 2026
@Davidson-Souza Davidson-Souza added the chore Cleaning, refactoring, reducing complexity label Feb 5, 2026
@Davidson-Souza
Copy link
Member Author

I'm open for suggestion regarding the struct's name 😂

A UtreexoProof struct contains three main items: a Merkle Proof,
the position targets and the preimage for each UTXO being spent.
RequestSelection allows you to choose which one of the structs
do you want independently.

This is serialized as a Bitmap, but the api uses a builder pattern
where you can use a setter to tell which ones do you want.

There's also the `request_all` helper that just sets all Bits, and
is useful to avoid having to call all three setters.
@csgui
Copy link
Contributor

csgui commented Feb 5, 2026

I'm open for suggestion regarding the struct's name 😂

Since it is related to bitmaps, how about ProofRequestMask? 🤷‍♂️

@luisschwab
Copy link
Member

UtreexoProofBitmask?

@Davidson-Souza
Copy link
Member Author

I've orbited away from *Bitmap because we already have something called Bitmap. But I guess it's hard to go somewhere else

@luisschwab
Copy link
Member

BitMASK

@JoseSK999
Copy link
Member

UtreexoProofFields or UtreexoProofMask?

@luisschwab luisschwab self-requested a review February 10, 2026 14:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Cleaning, refactoring, reducing complexity

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants