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.

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

Changes /ping route to return status code 200 instead of 204 when verbose is set #10321

Merged
merged 1 commit into from
Oct 2, 2018
Merged

Changes /ping route to return status code 200 instead of 204 when verbose is set #10321

merged 1 commit into from
Oct 2, 2018

Conversation

ryuhhnn
Copy link
Contributor

@ryuhhnn ryuhhnn commented Oct 1, 2018 •
edited
Loading

Closes #9772

Required for all non-trivial PRs
  • Sign CLA (if not already signed)

@ghost ghost added the proposed label Oct 1, 2018
@ghost ghost assigned jsternberg Oct 2, 2018
@ghost ghost added review and removed proposed labels Oct 2, 2018
@jsternberg jsternberg changed the title Changes /ping route to return status code 200 instead of 204. Changes /ping route to return status code 200 instead of 204 when verbose is set Oct 2, 2018
@jsternberg jsternberg merged commit e219d9f into influxdata:master Oct 2, 2018
@ghost ghost removed the review label Oct 2, 2018
@timhallinflux timhallinflux mentioned this pull request Nov 5, 2018
Closed
@krisero krisero mentioned this pull request Sep 30, 2020
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@jsternberg jsternberg

Labels
kind/feature-request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ryuhhnn @jsternberg