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

Removed expect script, Added -y to automate ubuntu installations #3

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

Removed expect script, Added -y to automate ubuntu installations #3

wants to merge 5 commits into from

Conversation

mukulhase
Copy link

  • Removes Expect Script.
  • Removes need to type the password every single time.
  • Up script can be used to notify the user when vpn connects.
  • Gets nameservers pushed by IIIT instead of hardcoded nameserver.

@mukulhase mukulhase force-pushed the master branch 2 times, most recently from ec4bb6f to f1d3779 Compare November 30, 2016 09:01
@bakszero
Copy link
Owner

bakszero commented Nov 30, 2016 •
edited
Loading

Add -e to echo in instances where you have used escape characters, otherwise in some systems, the \n is stored literally instead of adding a new line which throws up an error saying bad backslash in the linux_client.conf file.

@bakszero
Copy link
Owner

Not adding PR to master yet because Distros based on 14.04 Ubuntu and its derivates are not yet supported. Possibly waiting for an update on that issue from the openvpn github repository, and taking further action.

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

Successfully merging this pull request may close these issues.
2 participants
@mukulhase @bakszero