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

Beacon API peer_count call always returns 0 #3857

Closed
tdahar opened this issue Mar 14, 2022 · 2 comments · Fixed by #3860
Closed

Beacon API peer_count call always returns 0 #3857

tdahar opened this issue Mar 14, 2022 · 2 comments · Fixed by #3860
Assignees
Labels
good first issue Issues that are suitable for first-time contributors. prio-medium Resolve this some time soon (tm).

Comments

@tdahar
Copy link

tdahar commented Mar 14, 2022

While using the beacon API to get some data from the client, always peer_count call returns 0

curl localhost:9596/eth/v1/node/peer_count
{"data":{"disconnected":0,"connecting":0,"connected":0,"disconnecting":0}}

It would be expected to receive the current amount of peers in each state
curl localhost:9596/eth/v1/node/peer_count
{"data":{"disconnected":10,"connecting":9,"connected":30,"disconnecting":2}}

Running Lodestar with Docker, version 0.34.0

Screenshots

  • OS: Ubuntu
  • Version: 21.04
  • Image: chainsafe/lodestar:v0.34.0
@dadepo
Copy link
Contributor

dadepo commented Mar 14, 2022

Thanks for reporting this. The /eth/v1/node/peer_count endpoint is yet to be implemented. This is why it always returns 0. Once priorities allows, it would be implemented.

In the meantime, maybe you can check /eth/v1/node/peers? if it can be used as a workaround?

@philknows philknows added the good first issue Issues that are suitable for first-time contributors. label Mar 15, 2022
@dapplion
Copy link
Contributor

We should definitely implement this endpoint, sounds easy and aligns us with the api standard

@dapplion dapplion added the prio-medium Resolve this some time soon (tm). label Mar 15, 2022
@dadepo dadepo self-assigned this Mar 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
good first issue Issues that are suitable for first-time contributors. prio-medium Resolve this some time soon (tm).
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants