Skip to content

Commit

Permalink
Make api_key required and increase propogation seconds to recommended…
Browse files Browse the repository at this point in the history
… amount
  • Loading branch information
justafish committed Oct 18, 2024
1 parent a854561 commit d49be6c
Showing 1 changed file with 3 additions and 7 deletions.
10 changes: 3 additions & 7 deletions manifests/plugin/dns_linode.pp
Original file line number Diff line number Diff line change
Expand Up @@ -14,18 +14,14 @@
#
class letsencrypt::plugin::dns_linode (
Optional[String[1]] $package_name = undef,
Optional[String[1]] $api_key = undef,
String[1] $version = '4',
String[1] $api_key,
String[1] $version = '4',
Stdlib::Absolutepath $config_path = "${letsencrypt::config_dir}/dns-linode.ini",
Boolean $manage_package = true,
Integer $propagation_seconds = 10,
Integer $propagation_seconds = 120,
) {
include letsencrypt

if ! $api_key {
fail('No authentication method provided, please specify api_key and version.')
}

if $manage_package {
if ! $package_name {
fail('No package name provided for certbot dns linode plugin.')
Expand Down

0 comments on commit d49be6c

Please sign in to comment.