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

if ingress tls is enabled, then adjust the config file to overwriteprotocol https as needed so reverse proxy works #705

Open
lknite opened this issue Mar 6, 2025 · 0 comments
Labels
enhancement New feature or request

Comments

@lknite
Copy link

lknite commented Mar 6, 2025

Description of the change

Whatever changes are needed to config file to make a reverse proxy work should happen automatically if an ingress is enabled with tls.

Benefits

Folks deploying via helm chart should not have to know that they need to add a config section to get their reverse-proxy to work. It should be enough that they enabled an ingress with tls.

Possible drawbacks

None

Additional information

I had to add the following config to get the reverse-proxy to work, but every one deploying shouldn't have to dig this up every time, it should just be added automatically if an ingress was enabled with tls.

  nextcloud:

    # this is required or the reverse-proxy w/ ingress won't work
    configs:
      custom.config.php: |
        <?php
        $CONFIG = array (
          'overwriteprotocol' => 'https',
          'filelocking.enabled' => 'true',
          'enable_previews' => true,
        );

Since I'm not an expert I don't know if this config file overwrites default settings that I might actually need, or if it just appends these to an existing configuration. By just taking care of things for me, in response to enabling an ingress with tls, things will just work by folks who know the details and just take care of what needs to be taken care of automatically.

@lknite lknite added the enhancement New feature or request label Mar 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant