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

Fix incorrect bool expressions behavior due to params_lookup() behavi… #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

janbraiins
Copy link

…or and puppet 3.7+ changes

  • make string comparisons with a constant value of ''. The semantics
    of empty strings has changed in the later versions of puppet. Empty
    strings are now considered always true. Since params_lookup always
    returns an empty string for undef values, this causes a reverse
    logic behavior of the entire class.
  • change default value of fail2ban::noop parameter to false (not
    undef) for the same reasons as above. params_lookup would turn an
    undef to '', which in turn means true in puppet 3.7+

…or and puppet 3.7+ changes

- make string comparisons with a constant value of ''. The semantics
  of empty strings has changed in the later versions of puppet. Empty
  strings are now considered always true. Since params_lookup always
  returns an empty string for undef values, this causes a reverse
  logic behavior of the entire class.

- change default value of fail2ban::noop parameter to false (not
  undef) for the same reasons as above. params_lookup would turn an
  undef to '', which in turn means true in puppet 3.7+
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.

1 participant