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

Allow to define remote-cert-tls #266

Merged
merged 3 commits into from
Mar 6, 2018
Merged

Conversation

jkroepke
Copy link
Contributor

@jkroepke jkroepke commented Mar 6, 2018

https://community.openvpn.net/openvpn/wiki/DeprecatedOptions#a--ns-cert-type

ns-cert-type is deprecated in OpenVPN and will removed in the next version of OpenVPN.

The PR adds an opt-in for the backwards compatibility to use the new option.

@bastelfreak bastelfreak added the enhancement New feature or request label Mar 6, 2018
@bastelfreak
Copy link
Member

Hi @jkroepke, thanks for the PR. Can you take a look at the failing travis jobs and add a unit test?

@jkroepke
Copy link
Contributor Author

jkroepke commented Mar 6, 2018

@bastelfreak Done.

@bastelfreak
Copy link
Member

Thanks!

@bastelfreak bastelfreak merged commit 83b6d63 into voxpupuli:master Mar 6, 2018
@alexjfisher
Copy link
Member

In any of the supported OSes, does this module install any version of openvpn < 2.4?
ie Could the change be a bit more aggressive and just use the new option unless perhaps a fact says the old version is still installed? (I guess EPEL 6 didn't always have openvpn 2.4)

@jkroepke Thoughts?

@alexjfisher alexjfisher mentioned this pull request Mar 6, 2018
@jkroepke
Copy link
Contributor Author

jkroepke commented Mar 6, 2018

You support following OSes (https://github.com/voxpupuli/puppet-openvpn/blob/master/metadata.json#L11):

OS OVPN Version Link to package
Debian 7 2.2.1 https://packages.debian.org/wheezy/openvpn
Debian 8 2.3.4 https://packages.debian.org/jessie/openvpn
Debian 9 2.4.0 https://packages.debian.org/stretch/openvpn
Ubuntu 14.04 2.3.2 https://packages.ubuntu.com/trusty-updates/openvpn
Ubuntu 15.04 (Why? Its EOL) >2.3.2, <2.3.10 N/A
Ubuntu 16.04 2.3.10 https://packages.ubuntu.com/xenial-updates/openvpn
Ubuntu 18.04 (Upcomming) 2.4.4 https://packages.ubuntu.com/bionic/openvpn
RedHat/CentOS 6 2.4.4 https://rpmfind.net/linux/RPM/epel/6/x86_64/Packages/o/openvpn-2.4.4-1.el6.x86_64.html
RedHat/CentOS 7 2.4.4 https://rpmfind.net/linux/RPM/epel/7/x86_64/Packages/o/openvpn-2.4.4-1.el7.x86_64.html
Archlinux 2.4.5 https://www.archlinux.org/packages/core/x86_64/openvpn/
FreeBSD 2.4.4 https://www.freebsd.org/cgi/ports.cgi?query=openvpn&stype=all

remote-cert-tls is known since OpenVPN 2.1 you can switch to the new option from my side. But It's a breaking change.

Anyway, this modules generated client configs. Tunnelblick (macOS OpenVPN Client) will now use 2.5-git, if there are no deprecated options like comp-lzo and ns-cert-type.

@jkroepke jkroepke deleted the ns_cert_type branch March 7, 2018 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants