docker-pure-ftpd
docker-pure-ftpd copied to clipboard
My attempt to get anonymous access working
Got anonymous access working. Updated readme explaining that you need to omit the normal USER variables to setup the anonymous user
Hey @brandonmcclure Really sorry I missed this PR!
It looks fantastic, awesome work!
I think the only thing I think we should change before we can merge is instead of just omitting the existing env vars to assume anon access, we should instead have a dedicated FTP_ANONYMOUS_ACCESS=1 or something like that.
This way people have to purposeful opt into it as I use this container without the auto FTP env var based uses all the time too and wouldn't want this auto enabled.
What do you think?