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

Change nginx configs to also listen for ipv6 requests #264

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

flo-at
Copy link

@flo-at flo-at commented Sep 9, 2021

These changes to the nginx config files/templates make IPv6 work.

For IPv6-only setups this solves a more complex problem where you need to initially start the container with Let’s Encrypt disabled because it cannot get the chellenge file due to this missing listen directive in the nginx config. Then you would need to manually change the nginx config which is only generated at the initial start-up and therefore does not include the TLS config.

@robclancy
Copy link

It's crazy there is no response of this PR at all. I guess they only do things when they get support requests from paying customers.

@flo-at
Copy link
Author

flo-at commented Jan 10, 2023

@robclancy This is also true for other (bigger) problems in Seafile. If you are still using it, you might want to consider switching to something else. There are concerns that it's not safe anymore to expose Seafile on a public host. Also the repository is a hot mess.

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.

3 participants