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

Importing the generated .ovpn file into OpenVPN Connect mobile app fails #187

Closed
duckpuppy opened this issue Jan 13, 2016 · 7 comments
Closed

Comments

@duckpuppy
Copy link

The generated client .ovpn files have the following options set:

ca keys/<client-name>/ca.crt
cert keys/<client-name>/<client-name>.crt
key keys/<client-name>/<client-name>.key

Because the directives are present in the file, the mobile client will not import the .ovpn file due to the files not being present. Simply removing those three lines from the generated .ovpn file allows it to import successfully since those keys are also contained inline in the .ovpn file itself. If having those directives is necessary for some other reason, perhaps an option could be introduced to control the generation of those directives.

@luxflux
Copy link
Contributor

luxflux commented Mar 4, 2016

Unfortunately I am not a user of the .ovpn files. Maybe @elisiano can give us some insights?

@elisiano
Copy link
Contributor

elisiano commented Mar 4, 2016

Currently I'm using an older version of the module (prior to the ovpn refactoring). I'll try to update and report back.
Although if what @duckpuppy says is true, we have an issue. There should not be any external file reference (that's the whole point of having .ovpn files).

@elisiano
Copy link
Contributor

elisiano commented Mar 4, 2016

So yeah, this is the issue: https://github.com/luxflux/puppet-openvpn/blob/master/manifests/client.pp#L336-L340

It includes the original config in the ovpn file which contains references to the external files.

elisiano added a commit to elisiano/puppet-openvpn that referenced this issue Mar 4, 2016
Prior to this, .ovpn would include the .conf file, which means the external references to files were also included.
Now the main template has been split into 2 files: client.erb (included in both .conf and .ovpn) and client_external_auth.erb (included only in .conf)
This should fix issue voxpupuli#187
@elisiano
Copy link
Contributor

@duckpuppy did you have a chance to test the PR I submitted?

@duckpuppy
Copy link
Author

I haven't seen that a PR was submitted for this ticket. I should have some
time tomorrow to try it out.

On Thu, Mar 10, 2016 at 3:46 PM Elisiano Petrini [email protected]
wrote:

@duckpuppy https://github.com/duckpuppy did you have a chance to test
the PR I submitted?


Reply to this email directly or view it on GitHub
#187 (comment)
.

@luxflux
Copy link
Contributor

luxflux commented Mar 14, 2016

@duckpuppy did it work out?

luxflux pushed a commit that referenced this issue May 3, 2016
Prior to this, .ovpn would include the .conf file, which means the external references to files were also included.
Now the main template has been split into 2 files: client.erb (included in both .conf and .ovpn) and client_external_auth.erb (included only in .conf)
This should fix issue #187
@luxflux
Copy link
Contributor

luxflux commented May 3, 2016

I assume this is fixed by #190.

@luxflux luxflux closed this as completed May 3, 2016
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

No branches or pull requests

3 participants