Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update decred dependencies to latest master #213

Merged
merged 7 commits into from
May 16, 2024

Conversation

matheusd
Copy link
Member

This updates dcrd/dcrwallet dependencies to their latest master.

It also exposes the recently introduced --disablerelaytx and wallet birthday options.

This is currently a draft because it is waiting for some outstanding wallet PRs to be merged before the go.mod replacement directives can be removed.

matheusd added 7 commits May 13, 2024 16:17
Dcrwallet already requires go 1.20 as minimum version.
The sync status may change multiple times throughout the lifetime of the
wallet, so ensure only the first one is closing the channel.
This removes the initial part of the testFilterBlockDisconnected, which
causes a reorg all the way to genesis, in favor of only testing the
smaller reorg of an already setup chain.

The reorg to genesis will never actually happen in mainnet and SPV
wallets have some trouble dealing with large reorgs all the way to
genesis (specifically: the peers may get dropped due to too many
orphan/sidechain headers being sent).

In practice, decred reorgs are limited to a depth of 6 blocks due to PoS
validation, thus exercising a 10 block reorg is sufficient for testing
purposes.
The recently introduced disablerelaytx option from dcrwallet can be used
by SPV wallets to reduce their bandwidth and cpu consumptions, when the
users are not interested in mempool transactions.

This is useful in particular to mobile wallets, for which these
attributes are desirable.
This modifies the embedded wallet loader to pass along the aezeed
encoded wallet birthday to newly created wallets.

This offers a significant boost to the total setup time, in particular
for new wallets that have not yet been used (i.e. new seeds), for which
the initial account discovery and address discovery steps will now
complete much faster.

The birthday for new wallets has been modified to add a negative 24h
offset, to ensure no reorgs could make the wallet miss transactions.
@matheusd matheusd marked this pull request as ready for review May 16, 2024 17:20
@matheusd matheusd merged commit 7477c62 into decred:master May 16, 2024
9 checks passed
Copy link
Member

@davecgh davecgh left a comment

Choose a reason for hiding this comment

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

ACK

@matheusd matheusd deleted the update-dcrw branch May 16, 2024 18:16
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