fix: Convert from check.Script to check.Args for Consul API changes

gliderlabs / registrator

Service registry bridge for Docker with pluggable adapters

http://gliderlabs.com/registrator

MIT License

4.66k stars 912 forks source link

fix: Convert from check.Script to check.Args for Consul API changes #670

Closed ghost closed 2 years ago

ghost commented 4 years ago

This PR incorporates the changes from GH-643

"Consul 1.0.7 was the last version to support the string Script API, which had been long deprecated. 1.1.0 and later use []string Args. This PR updates Consul support to this new property."

I have compiled and tested and can confirm the Registrator SERVICE_*_CHECK_SCRIPT environment param works with Consol 1.6.1 with the below changes.

credit to @jhsolor