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

Forging fails when genesis block timestamp is in the future - Closes #6356 #6358

Merged
merged 4 commits into from
Apr 28, 2021

Conversation

ManuGowda
Copy link
Contributor

@ManuGowda ManuGowda commented Apr 27, 2021

What was the problem?

This PR resolves #6356

How was it solved?

  • Check for validator existence

How was it tested?

  • Added unit test
  • Tested manually by updating devnet genesis block timestamp to future(100sec)

@ManuGowda ManuGowda requested a review from shuse2 April 27, 2021 13:27
@ManuGowda ManuGowda self-assigned this Apr 27, 2021
framework/src/node/forger/forger.ts Outdated Show resolved Hide resolved
framework/test/unit/node/forger/forger.spec.ts Outdated Show resolved Hide resolved
@ManuGowda ManuGowda requested a review from shuse2 April 27, 2021 14:08
@ManuGowda ManuGowda merged commit d1c367c into hotfix/5.0.5 Apr 28, 2021
@ManuGowda ManuGowda deleted the 6356-bugfix-handle-future-forging branch April 28, 2021 09:23
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.

None yet

3 participants