Skip to content

Check already generated address when receiving mined tx#3178

Closed
guilhermemntt wants to merge 3 commits intodecred:masterfrom
guilhermemntt:guilhermemntt-check-already-generated-address
Closed

Check already generated address when receiving mined tx#3178
guilhermemntt wants to merge 3 commits intodecred:masterfrom
guilhermemntt:guilhermemntt-check-already-generated-address

Conversation

@guilhermemntt
Copy link
Contributor

@guilhermemntt guilhermemntt commented Jan 25, 2021

Depends on decred/dcrwallet#1967 and closes #3145.

This PR ensures that the current address stored in decrediton - nextAddress - is renewed if it was already used in some previous transaction and, in this way, strengthens the user's privacy. This checking is performed through getReceivedByAddress gRPC call to dcrwallet every time a new transaction is received.

@guilhermemntt guilhermemntt force-pushed the guilhermemntt-check-already-generated-address branch from 5bd813c to bdb2a42 Compare January 26, 2021 22:25
@guilhermemntt guilhermemntt marked this pull request as ready for review January 27, 2021 18:29
Check for already generated address only in current account
Simplify spent DCR checking
@guilhermemntt guilhermemntt force-pushed the guilhermemntt-check-already-generated-address branch from 7958668 to e5dc19d Compare March 4, 2021 05:47
@alexlyp
Copy link
Member

alexlyp commented Feb 1, 2022

Closing due to age. And not sure relying on a new dcrwallet request is the best way to solve this issue.

@alexlyp alexlyp closed this Feb 1, 2022
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.

check if new address already generated

2 participants