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

WebDav MOVE command fails #435

Closed
gabriele-v opened this issue Feb 16, 2025 · 3 comments
Closed

WebDav MOVE command fails #435

gabriele-v opened this issue Feb 16, 2025 · 3 comments

Comments

@gabriele-v
Copy link

Hello,
I've moved my binary Seafile 11 installation to Docker.

In my binary installation I've applied Nginx changes suggested here:
https://forum.seafile.com/t/seafdav-move-command-causing-502/11582/49

proxy_set_header   X-Forwarded-Proto $scheme;

Now in Docker I'm facing the same issue again.
SSL is not terminated on Nginx inside Seafile contatiner but on a Nginx Proxy Manager container that forwards in HTTP to Seafile container.

I've seen that in Seafile Nginx container the above line is present, I've removed it and everything works succesfully.
Considering that mar10/wsgidav#183 has been fixed and already added to Seafile 11 code, I don't think the above workaround is needed anymore.

Thanks!

@imwhatiam
Copy link
Member

I've seen that in Seafile Nginx container the above line is present,

Hello.

Are you saying that in your place, there is this line in the nginx configuration file used by Seafile docker?

proxy_set_header   X-Forwarded-Proto $scheme;

I just checked the nginx configuration template we use to create the Seafile docker image, there is not the above statement.

@gabriele-v
Copy link
Author

gabriele-v commented Feb 17, 2025

I'm quite sure that during migration I haven't changed Seafile container nginx configuration but only Seafile and Seahub configuration files. I'll try this evening to spin up a fresh container and see what happens.

EDIT: I think it has been removed here 735fe08 which only affects Seafile v12 images, while I've migrated a v11 installation

@imwhatiam
Copy link
Member

EDIT: I think it has been removed here 735fe08 which only affects Seafile v12 images, while I've migrated a v11 installation

It should be like this. In the latest version, there is no relevant configuration anymore.

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

No branches or pull requests

3 participants