Skip to content

Support all tracked sources for scanning utxo set and offer Get Addre…#528

Open
Kukks wants to merge 3 commits intobtcpayserver:masterfrom
Kukks:scan-utxo-group
Open

Support all tracked sources for scanning utxo set and offer Get Addre…#528
Kukks wants to merge 3 commits intobtcpayserver:masterfrom
Kukks:scan-utxo-group

Conversation

@Kukks
Copy link
Copy Markdown
Member

@Kukks Kukks commented Aug 25, 2025

…sses API

@NicolasDorier
Copy link
Copy Markdown
Member

NicolasDorier commented Aug 26, 2025

The problem is that there should not be a method that just pull all the addresses. That will end up blowing up on big wallets.
It should be streamed via keyset pagination.

It should also be split in 2 PR, the first with only GetAddress. This is already a big thing by itself.

@NicolasDorier
Copy link
Copy Markdown
Member

I think I already did the feedback on that, but writing it here, because I don't find it anymore:

If a group had a single derivation scheme, I would expect that scanning the group would be exactly the same thing as scanning the single derivation scheme.

However, this is not the case: The group will only scan the address that has been generated up to now, and will not attempt to scan beyond it.

Scanning a derivation scheme directly would try to find used addresses up to the max gap limit.

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