mirror of
https://github.com/hwdsl2/docker-ipsec-vpn-server.git
synced 2026-04-26 10:05:48 +03:00
[GH-ISSUE #48] Multi-user support #43
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Rillke on GitHub (Jan 2, 2018).
Original GitHub issue: https://github.com/hwdsl2/docker-ipsec-vpn-server/issues/48
In order to support multiple accounts, I had to customize the
CMD(run.sh) script either to allow the specification of multiple users or adding a check whether there are accounts in/etc/ppp/chap-secretsand/etc/ipsec.d/passwdbefore overwriting them so I could mount them as volumes and fill them with users using something similar toIs not supporting multiple users a design decision or would merge requests be accepted if they do not complicate things for a default setup.
@GabLeRoux commented on GitHub (Jan 4, 2018):
After reading the documentation a bit more, It doesn't seem possible to have multi user on first run (yet), but there is some documentation in the important notes:
avoid losing changes on container restart
I suppose the author is talking about these lines:
github.com/hwdsl2/docker-ipsec-vpn-server@38deb86170/run.sh (L164-L167)github.com/hwdsl2/docker-ipsec-vpn-server@38deb86170/run.sh (L202-L205)github.com/hwdsl2/docker-ipsec-vpn-server@38deb86170/run.sh (L207-L210)restart services
I suppose the only missing things from your script would be to restart the services once configurations are updated
@Rillke commented on GitHub (Jan 4, 2018):
Which is basically the same as restarting the entire container.
service ipsec restartdoes the same asreload)?@GabLeRoux commented on GitHub (Jan 4, 2018):
I did not try it yet, but I if I remember well,
reloadfornginxreloads configuration without killing the process and will keep running with older configuration in case of a wrong/broken configuration, preventing the service to go down, which is a good thing. So reloading is probably the right thing to do here instead.💩 that is not cool and I did not find anything online concerning this.
Maybe this is an other limitation? just like the limitation of the IPsec protocol, where multiple devices behind the same NAT (e.g. home router) cannot simultaneously connect to the VPN server unless using L2TP
@hwdsl2 commented on GitHub (Jan 4, 2018):
Hello! Adding new users may not require a restart of the ipsec service or
container. Feel free to test this without the restart command.