Skip to content
This repository has been archived by the owner on Jun 11, 2024. It is now read-only.

Fix invalid seed reveal end distance - Closes #6781 #6783

Merged
merged 3 commits into from
Sep 17, 2021

Conversation

ishantiw
Copy link
Contributor

What was the problem?

This PR resolves #6781

How was it solved?

  • By the end of distance in hash onion, Math.ceil was pointing to the previous checkpoint.

How was it tested?

npm run test:integration seed_reveal and run node test/test_app

- Incorrect next hash onion by the end of a distance of hash onion stored in config
@ishantiw ishantiw added this to the Sprint 53 milestone Sep 16, 2021
@ishantiw ishantiw self-assigned this Sep 16, 2021
@ishantiw ishantiw changed the base branch from development to hotfix/5.1.5 September 16, 2021 15:08
@shuse2 shuse2 removed this from the Sprint 53 milestone Sep 16, 2021
@shuse2 shuse2 merged commit 0d53083 into hotfix/5.1.5 Sep 17, 2021
@shuse2 shuse2 deleted the 6781-invalid-seed-reveal-end-distance branch September 17, 2021 07:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Invalid seed reveal is selected at the end of the hash onion section
3 participants