criteo / py-consul

Python client for Consul (http://www.consul.io/)
MIT License
45 stars 14 forks source link

Replace deprecated token query param by X-Consul-Token header #81

Closed mbrulatout closed 2 months ago

mbrulatout commented 2 months ago

Fix a pretty rare docker start flakyness by retrying a few times if required.

?token= query param is deprecated. It should have been removed in consul 1.17. While it's still properly handled as of 1.18 at least, we should now use X-Consul-Token header

This overrides a previous PR https://github.com/criteo/py-consul/pull/51