Skip to content
This repository has been archived by the owner on May 19, 2023. It is now read-only.

Add check for available Eth node #302

Closed
AuHau opened this issue Sep 18, 2020 · 0 comments · Fixed by #402
Closed

Add check for available Eth node #302

AuHau opened this issue Sep 18, 2020 · 0 comments · Fixed by #402
Assignees
@nduchak
Labels
difficulty: easy enhancement New feature or request good first issue Good for newcomers P1 - High

Comments

@AuHau
Copy link
Contributor

AuHau commented Sep 18, 2020

DevOps has rather often a problem when receiving error

Error: Invalid JSON RPC response: ""

While this is not issue of Cache but usually the deployment and most probably not accessible RSKj node, we should add a check to the ethFactory() which calls web3.eth.getProtocolVersion() that verifies that:

  • there is live server on the provider address
  • there is live ETH compatible server on the provider address

If the call fails, we can then display some better error about Provider's unavailability.
@AuHau AuHau added enhancement New feature or request good first issue Good for newcomers difficulty: easy P1 - High labels Sep 18, 2020
@nduchak nduchak linked a pull request Nov 18, 2020 that will close this issue
feat: add check for eth provider availability #402
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@nduchak nduchak

Labels
difficulty: easy enhancement New feature or request good first issue Good for newcomers P1 - High
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: add check for eth provider availability rsksmart/rif-marketplace-cache
2 participants
@AuHau @nduchak