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

Nginx configuration outdated (seafdav) #219

Closed
rsmuc opened this issue May 4, 2020 · 4 comments · May be fixed by #220
Closed

Nginx configuration outdated (seafdav) #219

rsmuc opened this issue May 4, 2020 · 4 comments · May be fixed by #220

Comments

@rsmuc
Copy link

rsmuc commented May 4, 2020

The configuration for nginx is outdated, so Seafdav is not working.
According the Upgrade-Notes Fastcgi is not supported anymore:
https://download.seafile.com/published/seafile-manual/upgrade/upgrade_notes_for_7.1.x.md

But the nginx configuration in the docker container still uses fastcgi. After replacing the nginx configuration with the mentioned snipped in the upgrade notes seafdav is working again.

rsmuc added a commit to rsmuc/seafile-docker that referenced this issue May 4, 2020
Fixed seafdav configuration 
fixes haiwen#219
@sndrsnk
Copy link

sndrsnk commented May 21, 2020

@rsmuc Is there any way I can manually add the solution outlined in your merge request to my SeaFile docker installation? I have a few services that are reliant on WebDav connections which are no longer working since I updated to 7.1. Thank you!

@rsmuc
Copy link
Author

rsmuc commented May 21, 2020

@rsmuc Is there any way I can manually add the solution outlined in your merge request to my SeaFile docker installation? I have a few services that are reliant on WebDav connections which are no longer working since I updated to 7.1. Thank you!

you can just modify the configuration file in the container

@sagebind
Copy link

I put the modified file on my file system and then used a volume mount to override the file used in the container.

@sndrsnk
Copy link

sndrsnk commented May 21, 2020

Worked great, thanks @rsmuc and @sagebind!

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 a pull request may close this issue.

4 participants