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

Added options for adding more consul configuration #503

Merged
merged 8 commits into from
Nov 25, 2015

Conversation

diptanu
Copy link
Contributor

@diptanu diptanu commented Nov 25, 2015

Fixed #484

Adds support for all the possible configuration that the consul api clients allows.

* `consul.verifyssl`: This option disables SSL verification when the transport
scheme for the Consul API client is `https`. This is set to true by default.

* `driver.whitelist`: A comma separated list of whitelisted drivers (e.g.
Copy link
Contributor

Choose a reason for hiding this comment

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

The whitelists shouldn't be here.

diptanu added a commit that referenced this pull request Nov 25, 2015
Added options for adding more consul configuration
@diptanu diptanu merged commit 836968c into master Nov 25, 2015
@diptanu diptanu deleted the f-consul-use-https branch November 25, 2015 23:14
@github-actions
Copy link

github-actions bot commented May 2, 2023

I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants