Skip to content

added functionality for optional certificate file and domain names #1

Open
azet wants to merge 1 commit intoUnyonsys:masterfrom
azet:master
Open

added functionality for optional certificate file and domain names #1
azet wants to merge 1 commit intoUnyonsys:masterfrom
azet:master

Conversation

@azet
Copy link

@azet azet commented Oct 10, 2012

site.pp - e.g.:
class { 'sssd':
domains => ['example.com'],
use_ecryptfs => false,
tls_ca_file => "example_certificate.cert",
tls_domain => "sub.example.com",
}

site.pp - e.g.:
  class { 'sssd':
    domains      => ['example.com'],
    use_ecryptfs => false,
    tls_ca_file  => "example_certificate.cert",
    tls_domain   => "sub.example.com",
  }
@brunoleon
Copy link
Contributor

This is using dynamic lookup which is not working in Puppet3.

I'll be happy to merge if you adress this.

@azet
Copy link
Author

azet commented Oct 24, 2012

Sorry, i'm pretty busy at the moment.

Compability issues are not my main concern ATM. I don't have time to fix all of your code.

@brunoleon
Copy link
Contributor

That's pretty rude answer when errors are in your own pull request...

@azet
Copy link
Author

azet commented Oct 25, 2012

I did not intend to sound rude at all, i apologize.

Please correct me if i'm wrong, but your module uses dynamic lookup too?
It is just a quick fix so one can supply certificates that do not follow a pre-defined naming scheme. i.e. "cert_example..", "ca_example.."

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.

2 participants