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

Fixed initial state panic for non genesis miners. #125

Merged
merged 1 commit into from
Mar 24, 2021
Merged

Conversation

Kenwes13
Copy link
Contributor

Premise

Right now the code panics if the initial_state.yaml isn't there. However, miners and sharpers joining the network have no need for the file to exist. The changes move the panic so only genesis miners will panic without the file.

Changes

  • When reading the initial state from the file the initStateErr is used for the error rather than err.
  • The initStateErr is used only if the node is in the genesis and causes a panic.

@Kenwes13 Kenwes13 merged commit af1e32f into master Mar 24, 2021
@platsko platsko deleted the initialState branch May 9, 2021 14:47
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.

None yet

1 participant