Skip to content
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.

Sign up for GitHub

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

Jump to bottom

changing users/logins via env variable not working #150

Open
mstenz opened this issue Dec 2, 2020 · 4 comments
Open

changing users/logins via env variable not working #150

mstenz opened this issue Dec 2, 2020 · 4 comments
Labels
question

Comments

@mstenz
Copy link

mstenz commented Dec 2, 2020

when allowing more users and logins via env variable this seems first to be applied when the servers starts up, but then in the end not be used and the default 5/5 users/logins are used when working with the server.
any idea?

Response: 421 5 users (the maximum) are already logged in, sorry
Error: Could not connect to server
@stilliard
Copy link
Owner

Hi @mstenz
When you're setting the FTP_MAX_CLIENTS, are you also setting the FTP_PASSIVE_PORTS to match?
These are tied together atm, more info here:
https://github.com/stilliard/docker-pure-ftpd#max-clients
Hope this helps

@mstenz
Copy link
Author

mstenz commented Dec 3, 2020

Hi,
i have not yet changed the passwive ports setting because i need to specify every port manually when creating the service, like

	--publish mode=host,target=21,published=21 \
	--publish mode=host,target=30000,published=30000 \
	--publish mode=host,target=30001,published=30001 \
	--publish mode=host,target=30002,published=30002 \

I am using docker service on swarm to create the container. using ranges seems not to be supported. Or do you have any idea how to easy do this?

@stilliard
Copy link
Owner

Hi @mstenz ah sorry I've not used docker swarm outside of a quick test.
Maybe they have a way to define ranges but you'd have to raise that with them I think, hopefully they have a solution for it rather than having lines for each port needed :)

@mstenz
Copy link
Author

mstenz commented Dec 3, 2020 •
edited
Loading

It seems, there is no way. At least when using mode=host the short way of doing it that support ranges is not allowed and gives back a syntax error.

Maybe you can adapt the documentation and add a swarm example, this is the one is used on my end now.

docker service create \
    --name pureftp \
    --mount type=bind,source=/storage/local/webroot,target=/ftp1 \
    --mount type=bind,source=/storage/local/pureftp/pure-ftpd,target=/etc/pure-ftpd \
    --mount type=bind,source=/storage/local/pureftp/log,target=/var/log/pure-ftpd \
    --publish mode=host,target=21,published=21 \
    --publish mode=host,target=30000,published=30000 \
    --publish mode=host,target=30001,published=30001 \
    --publish mode=host,target=30002,published=30002 \
    --publish mode=host,target=30003,published=30003 \
    --publish mode=host,target=30004,published=30004 \
    --publish mode=host,target=30005,published=30005 \
    --publish mode=host,target=30006,published=30006 \
    --publish mode=host,target=30007,published=30007 \
    --publish mode=host,target=30008,published=30008 \
    --publish mode=host,target=30009,published=30009 \
    --publish mode=host,target=30010,published=30010 \
    --publish mode=host,target=30011,published=30011 \
    --publish mode=host,target=30012,published=30012 \
    --publish mode=host,target=30013,published=30013 \
    --publish mode=host,target=30014,published=30014 \
    --publish mode=host,target=30015,published=30015 \
    --publish mode=host,target=30016,published=30016 \
    --publish mode=host,target=30017,published=30017 \
    --publish mode=host,target=30018,published=30018 \
    --publish mode=host,target=30019,published=30019 \
    --env PUBLICHOST=yourFQDN.com \
    --env ADDED_FLAGS="-d -d" \
    --env FTP_MAX_CLIENTS=20 \
    --env FTP_MAX_CONNECTIONS=20 \
    --env FTP_PASSIVE_PORTS=30000:30019 \
    stilliard/pure-ftpd:latest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
question
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@stilliard @mstenz