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 consul acl token note #1720

Merged
merged 1 commit into from
Jun 16, 2017
Merged

added consul acl token note #1720

merged 1 commit into from
Jun 16, 2017

Conversation

bitsofinfo
Copy link
Contributor

Documentation update on how to specify a consul acl token: #1717

## 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.
Copy link
Contributor

@nmengin nmengin Jun 14, 2017

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, done

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]"`
Copy link
Contributor

@nmengin nmengin Jun 14, 2017

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]"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure, done

@ldez ldez self-requested a review June 15, 2017 14:57
Copy link
Contributor

@nmengin nmengin left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@ldez ldez left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@emilevauge emilevauge left a comment

Choose a reason for hiding this comment

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

Thanks @bitsofinfo 👍
LGTM

@ldez ldez merged commit b452695 into traefik:master Jun 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants