Skip to content

Conversation

@mreiche
Copy link

@mreiche mreiche commented Dec 17, 2025

Description of the change

Fixes #825

@mreiche mreiche marked this pull request as ready for review December 18, 2025 07:43
Signed-off-by: mike.reiche <mike@reiche.world>
Signed-off-by: mike.reiche <mike@reiche.world>
Signed-off-by: mike.reiche <mike@reiche.world>
Signed-off-by: mike.reiche <mike@reiche.world>
@mreiche mreiche force-pushed the bugfix/nginx-redirect branch from 5f3c011 to 400eb26 Compare December 18, 2025 07:45
@wrenix
Copy link
Collaborator

wrenix commented Dec 18, 2025

@mreiche
Copy link
Author

mreiche commented Dec 18, 2025

Should we that submit upstream too?

https://github.com/nextcloud/documentation/blob/master/admin_manual/installation/nginx-root.conf.sample

No, because this setup has

server {
    listen 443 ssl http2;
    ...

So it's already running on the public port (443/https)

@mreiche
Copy link
Author

mreiche commented Dec 20, 2025

Can someone re-trigger the build jobs? They failed because of temporary problems.

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.

Nginx should respect the X-Forwarded-Proto header.

2 participants