-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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 consul acl token note #1720
Conversation
docs/user-guide/kv-config.md
Outdated
## Consul ACL Token support | ||
|
||
To specify a Consul ACL token for traefik to use you must set an system environment variable | ||
prior to launching traefik named `CONSUL_HTTP_TOKEN` equal to the acl token value. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe the sentence can looks like this : To specify a Consul ACL token for Traefik, we have to set a System Environment variable named **CONSUL_HTTP_TOKEN** prior to starting traefik. This variable must be initialized with the ACL token value.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure, done
docs/user-guide/kv-config.md
Outdated
To specify a Consul ACL token for traefik to use you must set an system environment variable | ||
prior to launching traefik named `CONSUL_HTTP_TOKEN` equal to the acl token value. | ||
|
||
If running traefik as a docker container you can do it like: `-e "CONSUL_HTTP_TOKEN=[consul-acl-token-value]"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Maybe the sentence can looks like this :
If Traefik is launched into a Docker container, the variable CONSUL_HTTP_TOKEN
can be initialized with the -e
Docker option : -e "CONSUL_HTTP_TOKEN=[consul-acl-token-value]"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure, done
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @bitsofinfo 👍
LGTM
Documentation update on how to specify a consul acl token: #1717