From e2a76b575b918ccd13edde17b4f81f12eae7d5e5 Mon Sep 17 00:00:00 2001 From: Michele Baldessari Date: Tue, 3 Feb 2026 10:20:40 +0100 Subject: [PATCH] Consider '' as a non-set value As reported by Cbutler, this makes little sense. So let's just consider empty strings as non-set values. --- plugins/module_utils/load_secrets_common.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/plugins/module_utils/load_secrets_common.py b/plugins/module_utils/load_secrets_common.py index f8ebe5d..9186736 100644 --- a/plugins/module_utils/load_secrets_common.py +++ b/plugins/module_utils/load_secrets_common.py @@ -312,13 +312,13 @@ def _validate_generate_mode(self, f): path = self._get_field_path(f) vault_policy = f.get("vaultPolicy", None) - if value is not None: + if value is not None and value != "": return ( False, "Secret has onMissingValue set to 'generate' but has a value set", ) - if path is not None: + if path is not None and path != "": return ( False, "Secret has onMissingValue set to 'generate' but has a path set",