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

Ping Endpoint Should Respect Auth Configuration #10167

Closed
jjathman opened this issue Aug 7, 2018 · 5 comments
Closed

Ping Endpoint Should Respect Auth Configuration #10167

jjathman opened this issue Aug 7, 2018 · 5 comments

Comments

@jjathman
Copy link

jjathman commented Aug 7, 2018

Proposal: Enable Auth for Ping Endpoint

Current behavior: Currently, the /ping endpoint will return successfully when Influx is running with HTTP authentication turned on even if the client does not provide the basic auth credentials.

Desired behavior: Either make the /ping endpoint respect the HTTP auth setting or allow a configuration setting to turn this on.

Use case: Spring Boot uses the Influx Java driver which currently uses the ping endpoint to determine the health of Influx. However, if authentication is enabled on Influx but the client does not have it configured or it isn't configured correctly any queries or writes will fail. Having a successful ping and health check provides a false indication that there aren't any configuration issues on the client. I would like at least for a configuration option to force the /ping endpoint to respect auth settings.

@stale
Copy link

stale bot commented Jul 24, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Jul 24, 2019
@jjathman
Copy link
Author

I would still like to see this done. The current behavior goes against what I think a users expectations are.

@stale stale bot removed the wontfix label Jul 24, 2019
@stale
Copy link

stale bot commented Oct 22, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the wontfix label Oct 22, 2019
@stale
Copy link

stale bot commented Oct 29, 2019

This issue has been automatically closed because it has not had recent activity. Please reopen if this issue is still important to you. Thank you for your contributions.

@jacobmarble
Copy link
Member

@jjathman this fix will land in 1.7.9.

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

No branches or pull requests

4 participants