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

Fail to proceed in listsinceblock if provided blockhash can't be found #1393

Merged
merged 1 commit into from Mar 13, 2017
Merged

Fail to proceed in listsinceblock if provided blockhash can't be found #1393

merged 1 commit into from Mar 13, 2017

Conversation

UdjinM6
Copy link

@UdjinM6 UdjinM6 commented Mar 13, 2017

If daemon crashes, it can't save latest block sometimes, so querying daemon
for presumably best/last hash would result in a list of all txes recognized by
this wallet as its own since genesis block which could be confusing,
to say at least. Same applies for typos etc. This should fix it.

Not sure why but such weird behaviour was the case since listsinceblock rpc was
initially introduced in Bitcoin 0.5 (Oct 5, 2011)
bitcoin@3a6e468

If daemon crashes, it can't save latest block sometimes, so querying daemon
for presumably best/last hash would result in a list of all txes recognized by
this wallet as its own since genesis block which could be confusing,
to say at least. Same applies for typos etc. This should fix it.

Not sure why but such weird behaviour was the case since listsinceblock rpc was
initially introduced in Bitcoin 0.5 (Oct 5, 2011)
bitcoin@3a6e468
@UdjinM6 UdjinM6 added the bug label Mar 13, 2017
@UdjinM6 UdjinM6 added this to the 12.1 milestone Mar 13, 2017
Copy link

@schinzelh schinzelh left a comment

Choose a reason for hiding this comment

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

utACK - no-brainer :)

@schinzelh schinzelh merged commit 6aaec3a into dashpay:v0.12.1.x Mar 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants