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

add ssl flag to convert scheme to https #3177

Merged
merged 1 commit into from
Jun 29, 2015

Conversation

jipperinbham
Copy link
Contributor

The following change adds an additional flag, -ssl to the influx command line in order to support https requests.

@otoolep
Copy link
Contributor

otoolep commented Jun 29, 2015

Thanks @jipperinbham -- isn't there more to this? Setting the path to the private key for example?

@jipperinbham
Copy link
Contributor Author

@otoolep I didn't think it'd be necessary as this PR is only to use https as the transport mechanism and not for certificate authentication.

My specific use case is running influxdb behind a reverse proxy that performs SSL termination.

@otoolep
Copy link
Contributor

otoolep commented Jun 29, 2015

Ah @jipperinbham -- this is the client ,not the server -- I missed that. I was wondering why it was so simple. :-)

@corylanou

@corylanou
Copy link
Contributor

+1

otoolep added a commit that referenced this pull request Jun 29, 2015
@otoolep otoolep merged commit bd3284f into influxdata:master Jun 29, 2015
@otoolep
Copy link
Contributor

otoolep commented Jun 29, 2015

Thanks @jipperinbham

@jipperinbham
Copy link
Contributor Author

awesome! thanks for getting this merged in so quickly.

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.

3 participants