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

feat: add support for insecure skip verify on cli http clients #15924

Merged
merged 1 commit into from
Nov 15, 2019

Conversation

jsteenb2
Copy link
Contributor

@jsteenb2 jsteenb2 commented Nov 14, 2019

Closes #15805

Adds support for insecure skip verify flag in influx cli.

Copy link
Contributor

@kelwang kelwang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, can double check, make sure, every service is covered?

@jsteenb2
Copy link
Contributor Author

@kelwang I added the change to every http service I could find, I'll double check again right now 👍

@jsteenb2
Copy link
Contributor Author

@kelwang good call, the Check service missed it. Everything should be covered now 👍

@jsteenb2 jsteenb2 merged commit ab50e18 into master Nov 15, 2019
@jsteenb2 jsteenb2 deleted the 15805/skip_cert_cli branch November 15, 2019 00:48
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 this pull request may close these issues.

Influx CLI should have the ability to skip cert validation
2 participants