Skip to content

[0.13.x] Change default value of spork15 to 1047200#2830

Merged
codablock merged 1 commit intodashpay:v0.13.xfrom
codablock:pr_v13_hardenspork15
Apr 4, 2019
Merged

[0.13.x] Change default value of spork15 to 1047200#2830
codablock merged 1 commit intodashpay:v0.13.xfrom
codablock:pr_v13_hardenspork15

Conversation

@codablock
Copy link

This is a hackish version of #2824,
meant for 0.13.x only. The reason for this hackish version is that the
code has diverged quite a bit and its not worth the effort to backport
the hardening code.

Even though 0.13.x included a lot of fixes for sporks handling and syncing,
I still feel more safe with hardening the spork15 block height. If
something goes wrong with spork syncing (e.g. its slower then the first
DIP2/3 block arrives), the whole sync process will fail otherwise.

This is a hackish version of dashpay#2824,
meant for 0.13.x only. The reason for this hackish version is that the
code has diverged quite a bit and its not worth the effort to backport
the hardening code.

Even though 0.13.x included a lot of fixes for sporks handling and syncing,
I still feel more safe with hardening the spork15 block height. If
something goes wrong with spork syncing (e.g. its slower then the first
DIP2/3 block arrives), the whole sync process will fail otherwise.
Copy link

@UdjinM6 UdjinM6 left a comment

Choose a reason for hiding this comment

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

utACK

@codablock codablock merged commit 53b2162 into dashpay:v0.13.x Apr 4, 2019
@codablock codablock deleted the pr_v13_hardenspork15 branch April 4, 2019 07:01
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