Merge pull request #225 from Pexers/main

Add support for Boolean extra-var interpretation
jenkinsci · Jun 18, 2024 · 8d0cadc · 8d0cadc
2 parents ae6975e + 65a1c42
commit 8d0cadc
Showing 1 changed file with 11 additions and 1 deletion.
diff --git a/src/main/java/org/jenkinsci/plugins/ansible/AbstractAnsibleInvocation.java b/src/main/java/org/jenkinsci/plugins/ansible/AbstractAnsibleInvocation.java
@@ -131,7 +131,17 @@ public ArgumentListBuilder appendExtraVars(ArgumentListBuilder args) {
                     value = Util.singleQuote(value);
                 }
                 StringBuilder sb = new StringBuilder();
-                sb.append(envVars.expand(var.getKey())).append("=").append(value);
+                // assuming Groovy representation for Boolean values
+                if (value.equals("true") || value.equals("false")) {
+                    // JSON format is required for Boolean variables
+                    sb.append("{\"")
+                            .append(envVars.expand(var.getKey()))
+                            .append("\":")
+                            .append(value)
+                            .append("}");
+                } else {
+                    sb.append(envVars.expand(var.getKey())).append("=").append(value);
+                }
                 if (var.isHidden()) {
                     args.addMasked(sb.toString());
                 } else {