From bfbd713536ed117b31a0cb1d5e03e51751298ebc Mon Sep 17 00:00:00 2001 From: John Walstra Date: Mon, 30 Jun 2025 17:54:48 -0500 Subject: [PATCH] For `pam action saas add`, if required field has default value, don't complain about missing value --- keepercommander/commands/pam_saas/add.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/keepercommander/commands/pam_saas/add.py b/keepercommander/commands/pam_saas/add.py index f8156f703..fce8df718 100644 --- a/keepercommander/commands/pam_saas/add.py +++ b/keepercommander/commands/pam_saas/add.py @@ -101,7 +101,7 @@ def execute(self, params: KeeperParams, **kwargs): # Make sure the SaaS configuration record has correct custom fields. missing_fields = [] for field in plugin.fields: - if field.required is True: + if field.required is True and field.default_value is None: found = next((x for x in config_record.custom if x.label == field.label), None) if not found: missing_fields.append(field.label.strip()) @@ -147,7 +147,7 @@ def execute(self, params: KeeperParams, **kwargs): return # If there is a resource record, it not NOOP. - # If there is NO resource record, it is NOOP.\ + # If there is NO resource record, it is NOOP. # However, if this is an IAM User, don't set the NOOP if acl.is_iam_user is False: acl.rotation_settings.noop = resource_uid is None