plugins: avoid circular dependencies

`include` the letsencrypt class instead of `require` it. Fixes #331.
voxpupuli · Oct 13, 2023 · ed36053 · ed36053
1 parent 19391cc
commit ed36053
Show file tree

Hide file tree

Showing 4 changed files with 4 additions and 4 deletions.
diff --git a/manifests/plugin/dns_cloudflare.pp b/manifests/plugin/dns_cloudflare.pp
@@ -23,7 +23,7 @@
   Boolean $manage_package           = true,
   Integer $propagation_seconds      = 10,
 ) {
-  require letsencrypt
+  include letsencrypt
 
   if ! $api_key and ! $api_token {
     fail('No authentication method provided, please specify either api_token or api_key and api_email.')

diff --git a/manifests/plugin/dns_rfc2136.pp b/manifests/plugin/dns_rfc2136.pp
@@ -24,7 +24,7 @@
   Stdlib::Absolutepath $config_dir = $letsencrypt::config_dir,
   Boolean $manage_package          = true,
 ) {
-  require letsencrypt
+  include letsencrypt
 
   if $manage_package {
     package { $package_name:

diff --git a/manifests/plugin/dns_route53.pp b/manifests/plugin/dns_route53.pp
@@ -12,7 +12,7 @@
   Integer $propagation_seconds     = 10,
   Boolean $manage_package          = true,
 ) {
-  require letsencrypt
+  include letsencrypt
 
   if $manage_package {
     package { $package_name:

diff --git a/manifests/plugin/nginx.pp b/manifests/plugin/nginx.pp
@@ -6,7 +6,7 @@
   Boolean $manage_package = true,
   String[1] $package_name = 'python3-certbot-nginx',
 ) {
-  require letsencrypt
+  include letsencrypt
 
   if $manage_package {
     package { $package_name: