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

/ext/health endpoint missing #26

Closed
Sanghren opened this issue May 31, 2020 · 2 comments · Fixed by #30
Closed

/ext/health endpoint missing #26

Sanghren opened this issue May 31, 2020 · 2 comments · Fixed by #30

Comments

@Sanghren
Copy link
Contributor

Sanghren commented May 31, 2020

Hello,

I noticed that /ext/health endpoint is missing in Slopes .

I'm using it for my small dashboard project and I would like to migrate all my calls via Slopes :)

Should I start a PR on this or was it left out intentionally ?

By the way this endpoint is not documented, I'll open an issue over there as well :)

Edit : Add an example of call / response :

curl -X POST --data '{
    "jsonrpc":"2.0",
    "id"     :1,
    "method" :"health.getLiveness"
}' -H 'content-type:application/json; http://localhost:9650/ext/health' 
{
  "jsonrpc": "2.0",
  "result": {
    "checks": {
      "network.validators.heartbeat": {
        "message": {
          "heartbeat": 1590918823
        },
        "timestamp": "2020-05-31T09:54:36.733208646Z",
        "duration": 7434,
        "contiguousFailures": 0,
        "timeOfFirstFailure": null
      }
    },
    "healthy": true
  },
  "id": 1
}
@collincusce
Copy link
Contributor

Thank you for pointing this out! We're happy to accept a PR, but I think you should collaborate in Discord with @cgcardona on this one as well. Let us know.

Related: https://github.com/ava-labs/docs/issues/24

@Sanghren Sanghren mentioned this issue Jun 4, 2020
2 tasks
@collincusce
Copy link
Contributor

Fixed with #30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants