search

atmoz / sftp

Securely share your files
https://hub.docker.com/r/atmoz/sftp/
MIT License
1.65k stars 834 forks source link

entrypoint shouldn't chown existing directories in /home #48

Closed ddrinka closed 8 years ago

ddrinka commented 8 years ago

I'm migrating from an existing sftp solution, and I have permissions the way I want them. I'm bringing in /home wholesale from a docker volume map. I don't want entrypoint to adjust permissions or ownership on any existing directories.

I'll do a quick hotpatch and pull request--take it or leave it, I'm happy to stay forked if your design pattern doesn't agree with my use case. :)

ddrinka commented 8 years ago

Ok, I take it all back. I didn't realize sshd required every home directory to be owned by root and unwritable. Your code is doing the correct thing, and apparently I will have to migrate my users to new writable subdirectories.

I'm closing this issue.