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

Fix redirect issue when docker is behind an other reverse proxy #158

Merged
merged 1 commit into from
Nov 7, 2024

Conversation

thelittlefireman
Copy link
Contributor

No description provided.

@thelittlefireman
Copy link
Contributor Author

thelittlefireman commented Jun 2, 2024

@crazy-max
Do you want me to edit something for getting this pr merged ?
regards

@thelittlefireman
Copy link
Contributor Author

up ?

@crazy-max
Copy link
Owner

Fix redirect issue when docker is behind an other reverse proxy

Can you clarify what the issue is? I prefer to keep relative path as default and avoid another round trip. If you want you can set SUBDIR=$scheme://$host. I'm closing this in the meantime but if you still have an problem, can you open an issue please? Thanks

@crazy-max crazy-max closed this Oct 5, 2024
@thelittlefireman
Copy link
Contributor Author

thelittlefireman commented Nov 7, 2024

@crazy-max
The issue is about well-known endpoints do not work behind a reverse proxy.

Cause 302 redirect to hostname and internal port and not 443 (frontal reverse proxy)

Thanks for the suggestion, i try. The internal nginx configuration looks good but due to this code : https://github.com/crazy-max/docker-nextcloud/blob/master/rootfs%2Fetc%2Fcont-init.d%2F03-config.sh#L222
M'y config.php is setup like this :

'overwritewebroot' => '$scheme://$host'

and so I'm redirected to https://nextcloud.xxxx.xx/$scheme://$host/login and so it's not working.

@crazy-max crazy-max reopened this Nov 7, 2024
@crazy-max
Copy link
Owner

The internal nginx configuration looks good but due to this code : https://github.com/crazy-max/docker-nextcloud/blob/master/rootfs%2Fetc%2Fcont-init.d%2F03-config.sh#L222 M'y config.php is setup like this :

'overwritewebroot' => '$scheme://$host'

Ah right forgot about overwritewebroot

@crazy-max crazy-max merged commit 83c75a6 into crazy-max:master Nov 7, 2024
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants