From 2fd869a5d4d91d75195119ff8badfc527531ee86 Mon Sep 17 00:00:00 2001 From: Thomas Bellman <bellman@lysator.liu.se> Date: Mon, 4 Oct 2021 17:43:45 +0200 Subject: [PATCH] Fix error handling bug in systemd::unit_options. The default case in the case switch for the $ensure parameter was incorrectly using the *string* "default" instead of the keyword ``default''. This meant that you did not get an error message if you specified an illegal value for that parameter (unless it happened to be exactly the string "default"). --- manifests/unit_options.pp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/manifests/unit_options.pp b/manifests/unit_options.pp index e933e0c..91fa8db 100644 --- a/manifests/unit_options.pp +++ b/manifests/unit_options.pp @@ -96,7 +96,7 @@ define systemd::unit_options( notify => Class['systemd::daemon_reload']; } } - 'default': { + default: { fail("Systemd::Unit_options[${title}]:", " Bad parameter ensure, ``${ensure}''") } -- GitLab