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

Bugfix when specifying CONFIG_FILE var #187

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

midzelis
Copy link
Contributor

@midzelis midzelis commented Jul 5, 2023

It appears that there was a CONFIG_FILE variable, but it was unused. The scripts always try to source conf from ./vpn.conf (so relative to the current working directory). Modify scripts so that if CONFIG_FILE is specified, it will use it, otherwise, it will fallback to using the ./vpn.conf from the current working directly.

Additionally, the instructions for wireguard mention a PreUp script, but its not in the example, so modified it.

This change should simplify a lot of the instructions in the README - you don't need to cd before executing the updown script - basically making most of these one liners, and executable from anywhere. However, I didn't update all the references in case you want to keep it as is.

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.

1 participant