You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
So the auto_trust_anchor_file variable is always set and always added to the config file. As soon as a trust_anchor_file is configured there are two "trust-anchor-file" variants in the config file.
I am not sure how you like to fix that: The cleanest option – from my point of view – would be, to make the auto_trust_anchor_file also a Optional and add some input validation, that not both values are set.
The text was updated successfully, but these errors were encountered:
b4ldr
added a commit
to b4ldr/puppet-unbound
that referenced
this issue
Oct 24, 2023
The config only supports having trust_anchor_file or
auto_trust_anchor_file. this updates the code to use trust_anchor_file
if it is set (its undef by default) otherwise use auto_trust_anchor_file
which has a default value.
fixesvoxpupuli#316
Affected Puppet, Ruby, OS and module versions/distributions
How to reproduce (e.g Puppet code you use)
What are you seeing
The
unbound.conf
cannot be parse anymore, because it contains aauto-trust-anchor-file:
and atrust-anchor-file:
option:What behaviour did you expect instead
Unbound should use my provided
trust_anchor_file
.Any additional information you'd like to impart
The part of the template that is responsible for this code is:
puppet-unbound/templates/unbound.conf.erb
Lines 175 to 176 in 825d42a
init.pp
:puppet-unbound/manifests/init.pp
Line 113 in 825d42a
puppet-unbound/manifests/init.pp
Line 180 in 825d42a
So the
auto_trust_anchor_file
variable is always set and always added to the config file. As soon as atrust_anchor_file
is configured there are two "trust-anchor-file" variants in the config file.I am not sure how you like to fix that: The cleanest option – from my point of view – would be, to make the
auto_trust_anchor_file
also aOptional
and add some input validation, that not both values are set.The text was updated successfully, but these errors were encountered: