-
Notifications
You must be signed in to change notification settings - Fork 59
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
fix: validator page validation vs ledger_closed race condition #917
Conversation
…to fix-vhs-logging-error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - just one typo
oh I thought it's okay to use abbreviation haha i've used indx in other code before |
Co-authored-by: Jackson Mills <aim4math@gmail.com>
It's not the end of the world, but it only saves a letter & can trip someone up when typing, so I figure writing it all out is better :) |
High Level Overview of Change
See issue #684
Context of Change
the validator page is stuck when retrieving info on a validator that has validated a ledger which doesn't have a state of validated or cannot be found, resulting in a "Something bad happened" page, see image:
Solution:
add error catching and log the error, return response anyway without last updated time and ledger index
Type of Change
Test Plan
added test in Validator.test.js