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

Fix RHEL 7 does not have openvpn-auth-ldap package #97

Closed
wants to merge 2 commits into from

Conversation

brunoleon
Copy link
Contributor

At the moment there is no openvpn-auth-ldap package under rhel 7
This fix allow the module to still install, though obviously the ldap feature won't be available

@luxflux
Copy link
Contributor

luxflux commented Oct 7, 2014

Thanks for your work!
Let's start with this Pull Request as all the others are branches of this one.

So RHEL 6 does have this package but RHEL 7 does not? Did its functionality move into another package? What about the easy-rsa package?

@brunoleon
Copy link
Contributor Author

Thanks for this modules ;-)
ATM there is no ldap for openvpn on RHEL7 as far as I know.
Easy-rsa is available from RHEL 6.4

@luxflux
Copy link
Contributor

luxflux commented Oct 17, 2014

Do you know whats the case for CentOS? Which version of it has which package?

@brunoleon
Copy link
Contributor Author

For these packages CentOS and RHEL are exactly the same.

@luxflux
Copy link
Contributor

luxflux commented Oct 28, 2014

Okay, so I think we should refactor this RHEL/CentOS part of params.pp.

Are the following statements correct?

  • CentOS doesn't have any support for LDAP authentication
  • RHEL doesn't have any support for LDAP authentication
  • CentOS < 6.4 doesn't have an easy-rsa package
  • CentOS >= 6.4 has an easy-rsa package
  • RHEL 6 doesn't have an easy-rsa package
  • RHEL 7 has an easy-rsa package

@brunoleon
Copy link
Contributor Author

This is correct but I think you do not need to differentiate RHEL from CentOS.
Just us $::osfamily and then narrow down choices per version.

@luxflux luxflux closed this in bae0c0e Jan 10, 2015
@luxflux
Copy link
Contributor

luxflux commented Jan 10, 2015

Finally got the time to fix this. Thank you for your patience.

typhonius pushed a commit to typhonius/puppet-openvpn that referenced this pull request Jan 11, 2015
@retizu
Copy link

retizu commented Oct 6, 2015

I've added some notes here that might be worth considering on the next iteration bae0c0e

@luxflux
Copy link
Contributor

luxflux commented Oct 10, 2015

There is already an issue for adding openvpn-auth-ldap: #172

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.

3 participants