-
Notifications
You must be signed in to change notification settings - Fork 90
env_vars: entrypoint.sh overrides environment variables #16
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
Comments
Suggestion
Then we can remove the The problem with this suggestion is that is becomes a "magic" env var which is only available when starting container with entrypoint. as such, exec into container - the env var will not be visible. Listen Port is significantly more difficult to fix, because that env var is used to determine which port should be exposed. |
Addresses: TykTechnologies#16 In order to not introduce breaking changes, legacy `TYKSECRET` takes precedence. If `TYK_GW_SECRET` is not set, then we need to hardcode the secret to ensure no issues if is missing from `tyk.conf`.
Addresses: TykTechnologies#16 Allow TYK_GW_SECRET to be used to set the secret in gateway. In order to not introduce breaking changes, legacy `TYKSECRET` takes precedence. If `TYK_GW_SECRET` is not set, then we need to hardcode the secret to ensure no issues if is missing from `tyk.conf`.
Addresses: #16 Allow TYK_GW_SECRET to be used to set the secret in gateway. In order to not introduce breaking changes, legacy `TYKSECRET` takes precedence. If `TYK_GW_SECRET` is not set, then we need to hardcode the secret to ensure no issues if is missing from `tyk.conf`.
Documentation indicates which env vars are available to set for the gateway
https://tyk.io/docs/configure/gateway-env-variables/
However entrypoint.sh overrides these variables
Either documentation should specify that if using standard docker installation, you need to set
TYKLISTENPORT
andTYKSECRET
rather thanTYK_GW_LISTENPORT
andTYK_GW_SECRET
.Or
entrypoint.sh should only set
TYK_GW_LISTENPORT
andTYK_GW_SECRET
if they are not already set.The text was updated successfully, but these errors were encountered: