Skip to content

Add ping endpoint #11191

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.

Sign up for GitHub

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

Jump to bottom
Closed
philwebb opened this issue Nov 28, 2017 · 3 comments
Closed

Add ping endpoint #11191

philwebb opened this issue Nov 28, 2017 · 3 comments
Labels
status: declined A suggestion or change that we don't feel we should currently apply type: enhancement A general enhancement

Comments

@philwebb
Copy link
Member

A general purpose ping endpoint that simply returns 200 could be useful.
@philwebb philwebb mentioned this issue Nov 29, 2017
Closed
@s-soroosh
Copy link

hi @philwebb,
I would be happy to work on this enhancement.
Could you please provide a bit more context about this issue?

@vpavic vpavic mentioned this issue Feb 2, 2018
Closed
@philwebb philwebb added for: team-attention An issue we'd like other members of the team to review and removed priority: normal labels Mar 21, 2018
@wilkinsona
Copy link
Member

wilkinsona commented Mar 28, 2018 •
edited
Loading

I'm not sure about this one. The benefit over anonymously accessing the health endpoint seems pretty small. I wonder if there's any commonly used infrastructure that would use both /ping and /health and make use of the possibly different status code that each would return?

@snicoll
Copy link
Member

snicoll commented Mar 28, 2018

Didn't we create this issue due to the initial design of the health endpoint in 2.0? Now that this has been revisited and we have a proper strategy, I am not sure this issue makes more sense.

@philwebb philwebb added status: declined A suggestion or change that we don't feel we should currently apply and removed for: team-attention An issue we'd like other members of the team to review labels Mar 28, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
status: declined A suggestion or change that we don't feel we should currently apply type: enhancement A general enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@snicoll @philwebb @s-soroosh @wilkinsona