-
Notifications
You must be signed in to change notification settings - Fork 158
add keycloak example #465
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
add keycloak example #465
Conversation
|
@rhafer what do you think? Maybe add this to the opencloud_full example? |
Yes. Let's try that. |
then I'll add it later (after release is done 😀) |
please try to delete this line https://github.com/opencloud-eu/opencloud/pull/465/files#diff-40bc490c0c8e661bd131829ba24af60db667dda9fe7c5e3566433a95a6e53666R83 and start again |
|
Hi, After testing, I found that the following line in the csp.yaml file of the opencloud_full example: - 'https://${KEYCLOAK_DOMAIN|keycloak.opencloud.test}/'does not work as expected because the docker-compose file for OpenCloud does not pass the environment variable. As a result, even if KEYCLOAK_DOMAIN is defined in the .env file, the CSP configuration still falls back to keycloak.opencloud.test due to the missing environment variable. To resolve this, I added the following lines to keycloak.yml: opencloud:
environment:
KEYCLOAK_DOMAIN: ${KEYCLOAK_DOMAIN:-keycloak.opencloud.test} I'm not sure if this is the best approach. Should the original example also be updated accordingly? |
Thank you very much. Your approach is working perfectly 😍. I've created a fix |
…kYaml fix keycloak example #465
Steps:
cd deployments/examples/opencloud_fullenable keycloak in the
.envfile https://github.com/opencloud-eu/opencloud/pull/465/files#diff-57b97469284982c33a6c68511b9b7524e5142e7bfce6867b1186b457f74a0c56R242OC_DOCKER_IMAGE=opencloudeu/opencloud OC_DOCKER_TAG=2.0.0 docker compose up -dadd
127.0.0.1 keycloak.opencloud.testto/etc/hostsopen https://keycloak.opencloud.test/ admin:admin
open https://cloud.opencloud.test/ admin:admin