Refactor sudo template

Author: silug

manifests/service/sudo.pp

@@ -40,15 +40,49 @@
       },
     )
   } else {
+    # Build configuration content for the SUDO service
+    $_base_content = [
+      '# sssd::service::sudo',
+      '[sudo]',
+    ]
+
+    # Add conditional parameters if defined
+    if $description {
+      $_description_entries = ["description = ${description}"]
+    } else {
+      $_description_entries = []
+    }
+
+    if $debug_level {
+      $_debug_level_entries = ["debug_level = ${debug_level}"]
+    } else {
+      $_debug_level_entries = []
+    }
+
+    $_debug_timestamps_entries = $debug_timestamps ? {
+      true  => ['debug_timestamps = true'],
+      false => ['debug_timestamps = false'],
+    }
+
+    $_debug_microseconds_entries = $debug_microseconds ? {
+      true  => ['debug_microseconds = true'],
+      false => ['debug_microseconds = false'],
+    }
+
+    $_sudo_timed_entries = $sudo_timed ? {
+      true  => ['sudo_timed = true'],
+      false => ['sudo_timed = false'],
+    }
+
+    # Combine all configuration entries in the expected order
+    $_all_entries = $_base_content + $_description_entries + $_debug_level_entries + $_debug_timestamps_entries + $_debug_microseconds_entries + $_sudo_timed_entries
+
+    $_final_content = $_all_entries.join("\n")
+
     $_content = epp(
       "${module_name}/service/sudo.epp",
       {
-        'description'        => $description,
-        'debug_level'        => $debug_level,
-        'debug_timestamps'   => $debug_timestamps,
-        'debug_microseconds' => $debug_microseconds,
-        'sudo_timed'         => $sudo_timed,
-        'sudo_threshold'     => $sudo_threshold,
+        'content' => $_final_content,
       },
     )
   }

templates/service/sudo.epp

@@ -1,20 +1,4 @@
-<% |
-  Optional[String]            $description,
-  Optional[Sssd::Debuglevel]  $debug_level,
-  Boolean                     $debug_timestamps,
-  Boolean                     $debug_microseconds,
-  Boolean                     $sudo_timed,
-  Integer[1]                  $sudo_threshold,
+<%- |
+  String $content,
 | -%>
-
-# sssd::service::sudo
-[sudo]
-<% if $description { -%>
-description = <%= $description %>
-<% } -%>
-<% if $debug_level { -%>
-debug_level = <%= $debug_level %>
-<% } -%>
-debug_timestamps = <%= $debug_timestamps %>
-debug_microseconds = <%= $debug_microseconds %>
-sudo_timed = <%= $sudo_timed %>
+<%= $content %>