Skip to content

Update node health endpoint #4292

@michaelsproul

Description

@michaelsproul

Description

As suggested by @jmcruz1983 here we should update the /eth/v1/node/health endpoint to account for the execution node's sync status.

I think we should change the semantics to return syncing if:

let unhealthy = syncing || el_offline || optimistic

Currently it's just checking the BN's sync status.

Additional Info

Related to #4291, which will add the el_offline field.

Metadata

Metadata

Assignees

Labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions