keycloak / keycloak-quickstarts

Apache License 2.0
1.98k stars 984 forks source link

Remove KC_PROXY for Kubernetes #589

Closed jamesshuriff closed 1 month ago

jamesshuriff commented 2 months ago

Replaced KC_PROXY with KC_HTTP_ENABLED for Kubernetes

Closes #557

codespearhead commented 1 month ago

Thanks for the PR, but it's not the right solution.

If you want to run Keycloak in production mode behind a reverse proxy, you need to inform Keycloak that it's behind one [1]. See this answer [1][2].

Could you close this PR in favor of [3]?

[1] https://github.com/keycloak/keycloak/issues/33414 [2] https://github.com/keycloak/keycloak/issues/33414#issuecomment-2389657317 [3] https://github.com/keycloak/keycloak-quickstarts/pull/612

mposolda commented 1 month ago

@jamesshuriff Closing as suggested by @codespearhead . Please re-open if you think that this should be the preferred approach.