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

Replace Consul HTTP Client with Interface Clients #840

Draft
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

M-Whitaker
Copy link

As stated in #475, Consul token query parameter deprecated in Consul 1.17 however the current Consul HTTP client seems to be no longer maintained.

Moving to interface clients also has the added benefit for allowing for blocking and non-blocking clients to be used.

Further to this change, the docker image used for integration testing this lib is using an outdated image: https://hub.docker.com/_/consul and does not have the latest consul images so this needed to be moved to https://hub.docker.com/r/hashicorp/consul.

Any suggestions are VERY welcome, I assume this only as a first draft for this change so have only migrated the config module at this moment.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants