Skip to content
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 trusted_domains after installation #640

Open
denppa opened this issue Oct 2, 2024 · 3 comments
Open

Add trusted_domains after installation #640

denppa opened this issue Oct 2, 2024 · 3 comments
Labels
documentation Improvements or additions to documentation good first issue Small tasks with clear documentation about how and in which place you need to fix things in.

Comments

@denppa
Copy link

denppa commented Oct 2, 2024

Installed Nextcloud via helm on a baremetal k8s cluster. Created an ingress from another domain to point to Nextcloud service, but it won't let me visit the site because of a trusted_domain error.

Went back into values.yml and added the new domain, ran:

helm upgrade -n nextcloud nextcloud/nextcloud --values values.yml

However, the config.php did not reflect the new list of trusted domains after restart.

Relevent values.yml

  trustedDomains:
    - nextcloud.mydomain.com
    - nextcloud.mydomain2.com

mydomain2.com is the new domain, did not get added to config.php.

@jessebot
Copy link
Collaborator

hmmm, not sure if the way we set the trusted domain is parsed a second time after installation, but can you try running the occ command to update it?

It should be something like:

kubectl exec -it your-nextcloud-pod -- /bin/sh -c "php occ config:system:set trusted_domains 2 --value=nextcloud.mydomain2.com"

@denppa
Copy link
Author

denppa commented Oct 20, 2024

hmmm, not sure if the way we set the trusted domain is parsed a second time after installation, but can you try running the occ command to update it?

It should be something like:

kubectl exec -it your-nextcloud-pod -- /bin/sh -c "php occ config:system:set trusted_domains 2 --value=nextcloud.mydomain2.com"

This is the solution. Would be great if the documentation can mention that certain params inside values.yml will need to be changed manually after deployment.

@jessebot
Copy link
Collaborator

This is the solution.

Ok, good, glad it's at least not a blocker 🙏

Would be great if the documentation can mention that certain params inside values.yml will need to be changed manually after deployment.

Sure, we're open to PRs for this in the README here:
https://github.com/nextcloud/helm/tree/main/charts/nextcloud

@jessebot jessebot added good first issue Small tasks with clear documentation about how and in which place you need to fix things in. documentation Improvements or additions to documentation labels Oct 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation good first issue Small tasks with clear documentation about how and in which place you need to fix things in.
Projects
None yet
Development

No branches or pull requests

2 participants