Thanks to visit codestin.com
Credit goes to github.com

Skip to content

add keycloak example#465

Merged
v-scharf merged 5 commits into
mainfrom
addKeycloakExample
Mar 28, 2025
Merged

add keycloak example#465
v-scharf merged 5 commits into
mainfrom
addKeycloakExample

Conversation

@v-scharf

@v-scharf v-scharf commented Mar 24, 2025

Copy link
Copy Markdown
Contributor

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

@v-scharf v-scharf requested review from micbar and rhafer March 24, 2025 16:09
@micbar

micbar commented Mar 24, 2025

Copy link
Copy Markdown
Member

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

@rhafer

rhafer commented Mar 25, 2025

Copy link
Copy Markdown
Member

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

Yes. Let's try that.

@v-scharf

v-scharf commented Mar 25, 2025

Copy link
Copy Markdown
Contributor Author

Yes. Let's try that.

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

@v-scharf v-scharf mentioned this pull request Mar 25, 2025
50 tasks
@amrita-shrestha

amrita-shrestha commented Mar 25, 2025

Copy link
Copy Markdown
Contributor

@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

@v-scharf v-scharf added the Type:DevOps Related to CI/CD label Mar 27, 2025
@v-scharf

Copy link
Copy Markdown
Contributor Author

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

@v-scharf v-scharf merged commit 86354e6 into main Mar 28, 2025
@v-scharf v-scharf deleted the addKeycloakExample branch March 28, 2025 14:24
@HiFallMaple

Copy link
Copy Markdown

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?

v-scharf added a commit that referenced this pull request Mar 31, 2025
@v-scharf

Copy link
Copy Markdown
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

v-scharf 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.

5 participants