search

fauria / docker-vsftpd

🐳 vsftpd Docker image based on Centos 7. Supports passive mode and virtual users.
https://hub.docker.com/r/fauria/vsftpd/
Apache License 2.0
414 stars 341 forks source link

FTP_USER entry now appended to virtual_users.txt instead of overwriting the whole file #55

Open gregory112 opened 4 years ago

gregory112 commented 4 years ago

Check the virtual_users.txt file for FTP_USER entry first, to see if the entry already exists. If the user already exists, it won't be appended to the file. Using ">>" instead of ">" stream which will overwrite the file.

evantill commented 4 years ago

+1 for this PR

fauria commented 3 years ago

Thanks for the PR, @gregory112 !

We can try to find a way for appending virtual users, but the proposed grep approach wont work, as it will behave in an unexpected way if it matches a password.

yale8848 commented 3 years ago

fix this issue here: https://github.com/yale8848/docker-vsftpd