-
Notifications
You must be signed in to change notification settings - Fork 2
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
add with-peruserlimits
#7
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@matthewh86 thanks for the PR!
I'm not against --withperuserlimits
and --localstatedir=/var/pure-ftpd
.
However, I would like not to change --sysconfdir=/etc/pure-ftpd
since it would break existing installations using default paths quite hard. You still can mount anything in one dir just by specifying custom paths as described in README.
Also, I wonder: why would you want to mount /var/pure-ftpd
? Judging from default installations, it only writes /var/run/pure-ftpd.pid
, since /var/log/
is written to syslog
. What would be the case to preserve .pid
file between container restarts?
Regarding the pid question for |
with-peruserlimits
In case you were interested, here's the docker image I spent way too long fighting with to get to state where I'm mostly happy with how it functions. https://github.com/noxtech/anonymous-pure-ftp
Things which gave me a headache were how docker mounts volumes, screws with the file/directory permissions and ownership, and various other quirks that I've forgotten about. Here's the docker-compose.yml entry:
|
Yeah... it's better on Kubernetes. But Docker Compose may give some headache here. Nevertheless, the image comes with |
Released as 1.0.51-r28. |
hi, thanks for the base image!
this separates out the configuration from the image'sremoved as discussed/etc
and/var
locations to allow for simpler volume mounting (e.g. for persisting configuration and user accounts).this adds with-peruserlimits to support the PER_USER_LIMITS property that is useful for allowing more than 20 anonymous connections