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

enable single config file support #159

Merged
merged 4 commits into from May 21, 2023

Conversation

zilchms
Copy link
Contributor

@zilchms zilchms commented May 3, 2023

Fixes some parts of #158 .
Defines the openssl::config type which handles exactly one config file per resource instantiation.
Changes the openss::configs class to use the new type.

Allows other modules to instantiate a config file as follows:

openssl::config {'/path/to/config':
  ensure     => 'present',
  commonname => 'somewhere.org'
}

#
define openssl::config (
Variant[String, Array[String]] $commonname,
String $country,
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for all the string parameters, please check if empty strings make sense, otherwise switch to String[1]

@bastelfreak bastelfreak merged commit 3b65cbb into voxpupuli:master May 21, 2023
3 checks passed
@zilchms zilchms deleted the introduce-config-type branch May 21, 2023 17:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants