delfer / docker-alpine-ftp-server

Small and flexible docker image with vsftpd server

186 stars 131 forks source link

Open moinologics opened 10 months ago

moinologics commented 10 months ago

i see VOLUME /ftp/ftp in dockerfile, is it required??

delfer commented 10 months ago

@moinologics hello! https://docs.docker.com/engine/reference/builder/#volume No, it's not required. Do you have any issues with this?

moinologics commented 10 months ago

Yes, it's mount this volume even if we don't want to, so it will be good if we can remove

delfer commented 10 months ago

@moinologics can you please provide a PR? I would glad to merge it!

QuiGonLeong commented 1 month ago

@delfer PR created to remove the VOLUME /ftp/ftp

74