Skip to content

Conversation

@ScharfViktor
Copy link
Contributor

@ScharfViktor ScharfViktor commented Mar 24, 2025

Steps:
cd deployments/examples/opencloud_full
enable keycloak in the .env file https://github.com/opencloud-eu/opencloud/pull/465/files#diff-57b97469284982c33a6c68511b9b7524e5142e7bfce6867b1186b457f74a0c56R242
OC_DOCKER_IMAGE=opencloudeu/opencloud OC_DOCKER_TAG=2.0.0 docker compose up -d

add 127.0.0.1 keycloak.opencloud.test to /etc/hosts
open https://keycloak.opencloud.test/ admin:admin

image

open https://cloud.opencloud.test/ admin:admin

@ScharfViktor ScharfViktor requested review from micbar and rhafer March 24, 2025 16:09
@micbar
Copy link
Contributor

micbar commented Mar 24, 2025

@rhafer what do you think? Maybe add this to the opencloud_full example?

@rhafer
Copy link
Contributor

rhafer commented Mar 25, 2025

@rhafer what do you think? Maybe add this to the opencloud_full example?

Yes. Let's try that.

@ScharfViktor
Copy link
Contributor Author

ScharfViktor commented Mar 25, 2025

Yes. Let's try that.

then I'll add it later (after release is done 😀)

@ScharfViktor ScharfViktor mentioned this pull request Mar 25, 2025
50 tasks
@amrita-shrestha
Copy link
Contributor

amrita-shrestha commented Mar 25, 2025

@ScharfViktor, everything i try to login to opencloud. I came across this error

please try to delete this line https://github.com/opencloud-eu/opencloud/pull/465/files#diff-40bc490c0c8e661bd131829ba24af60db667dda9fe7c5e3566433a95a6e53666R83 and start again

@ScharfViktor ScharfViktor added the Type:DevOps Related to CI/CD label Mar 27, 2025
@ScharfViktor
Copy link
Contributor Author

@micbar @rhafer I have added keycloak to opencloud_full
works fine for me. please have look

@ScharfViktor ScharfViktor merged commit 86354e6 into main Mar 28, 2025
50 checks passed
@ScharfViktor ScharfViktor deleted the addKeycloakExample branch March 28, 2025 14:24
@HiFallMaple
Copy link

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?

ScharfViktor added a commit that referenced this pull request Mar 31, 2025
@ScharfViktor
Copy link
Contributor Author

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

ScharfViktor added a commit that referenced this pull request Mar 31, 2025
@openclouders openclouders mentioned this pull request Mar 28, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type:DevOps Related to CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants