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.

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

add Dockerfile and notes #1074

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

inceabdullah
Copy link

Hello,

I generally use openfortivpn with docker. There are some fortivpnclient docker images but not openfortivpn. So I would like to add Dockerfle to use it everywhere.

in Docker needed two param --privileged --cap-add=NET_ADMIN and --net host

There two options to run it

docker run --privileged --cap-add=NET_ADMIN --net host -d -v config:/etc/openfortivpn/config openforticlient

or could be run with other params like --set-dns=0 --pppd-use-peerdns=1

docker run --privileged --cap-add=NET_ADMIN --net host -d -v config:/etc/openfortivpn/config openforticlient -c /etc/openfortivpn/config --set-dns=0 --pppd-use-peerdns=1

Thank you.

@adrienverge
Copy link
Owner

Hello,

Having a Docker image for openfortivpn is a good idea, but like for distribution packages (rpm, deb, flatpak, etc.) I believe it's better not to maintain them here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants