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

Windows clients integration #19

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

Windows clients integration #19

wants to merge 18 commits into from

Conversation

fitz123
Copy link

@fitz123 fitz123 commented Oct 21, 2015

Hi there!

I understand that current pull request is ugly, too many changes, but If you accept my concept I can change it, separate and so on.

Basically my improvements are:

  1. Security - at the end you have password protected key
  2. Copy key on "builders" pc. At least my use case is generate new users and get the keys for further distribution
  3. Using *PKCS cannot read: /etc/openvpn/keys/crl.pem: Permission denied (errno=13) #12 format for keys, because is allow end users upload key in Windows keychain

At the end I have fully automated configuration with secured keys.

Take a look at windows installation guide
This is automatically connection during boot time, not required admin access to use, not required password to connect (but cert itself is password protected)

So up to you

@fitz123
Copy link
Author

fitz123 commented Oct 21, 2015

I really don't want to support my own fork, so please say that changes make sense so I can do it properly and make if pass Travis tests

@klen
Copy link
Member

klen commented Oct 28, 2015

Of course, it makes sense, but I need my time to check the PR. I'll try to do as I can.

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