linuxserver / docker-swag

Nginx webserver and reverse proxy with php support and a built-in Certbot (Let's Encrypt) client. It also contains fail2ban for intrusion prevention.
https://docs.linuxserver.io/general/swag
GNU General Public License v3.0
2.81k stars 242 forks source link

[FEAT] Consider proxy_hide_header option for Safari browsers accessing Nextcloud #446

Closed jakev383 closed 5 months ago

jakev383 commented 8 months ago

Is this a new feature request?

Wanted change

Add the following option to the nextcloud sample configs: proxy_hide_header Upgrade;

Reason for change

Safari browsers (both mobile and desktop) seem to encounter an issue accessing Nextcloud proxied instances. I have not seen this behavior exhibited for other proxied containers (mealie, emby, etc). The Nextcloud destination page/site fails to load and the client displays the following error: network connection is lost I have only seen this with Safari browsers so far - Chrome and Firefox do not seem to need this option to function as expected, nor does it seem to affect Chrome or Firefox after being implemented. Adding the config change to the nextcloud.subdomain.config resolved the issue.

Proposed code change

Add the following to the nextcloud.sub*.config.sample files: proxy_hide_header Upgrade;

github-actions[bot] commented 8 months ago

Thanks for opening your first issue here! Be sure to follow the relevant issue templates, or risk having this issue marked as invalid.

LinuxServer-CI commented 7 months ago

This issue has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.

github-actions[bot] commented 4 months ago

This issue is locked due to inactivity