chore: fix missing sentinel getattr

heitorlessa · heitorlessa · commit 4ec3c7ee230a · 2024-02-19T11:47:59.000+01:00
diff --git a/aws_lambda_powertools/utilities/feature_flags/schema.py b/aws_lambda_powertools/utilities/feature_flags/schema.py
@@ -355,7 +355,7 @@ def validate_condition_key(condition: Dict[str, Any], rule_name: str):
         #
         # maintenance: we should split to separate file/classes for better organization, e.g., visitor pattern.
 
-        custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_key")
+        custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_key", None)
 
         # ~90% of actions available don't require a custom validator
         # logging a debug statement for no-match will increase CPU cycles for most customers
@@ -381,7 +381,7 @@ def validate_condition_value(condition: Dict[str, Any], rule_name: str):
         #
         # maintenance: we should split to separate file/classes for better organization, e.g., visitor pattern.
 
-        custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_value")
+        custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_value", None)
 
         # ~90% of actions available don't require a custom validator
         # logging a debug statement for no-match will increase CPU cycles for most customers

Original file line number	Diff line number	Diff line change
`@@ -355,7 +355,7 @@ def validate_condition_key(condition: Dict[str, Any], rule_name: str):`
`355`	`355`	`#`
`356`	`356`	`# maintenance: we should split to separate file/classes for better organization, e.g., visitor pattern.`
`357`	`357`
`358`		`- custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_key")`
	`358`	`+ custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_key", None)`
`359`	`359`
`360`	`360`	`# ~90% of actions available don't require a custom validator`
`361`	`361`	`# logging a debug statement for no-match will increase CPU cycles for most customers`
`@@ -381,7 +381,7 @@ def validate_condition_value(condition: Dict[str, Any], rule_name: str):`
`381`	`381`	`#`
`382`	`382`	`# maintenance: we should split to separate file/classes for better organization, e.g., visitor pattern.`
`383`	`383`
`384`		`- custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_value")`
	`384`	`+ custom_validator = getattr(ConditionsValidator, f"_validate_{action.lower()}_value", None)`
`385`	`385`
`386`	`386`	`# ~90% of actions available don't require a custom validator`
`387`	`387`	`# logging a debug statement for no-match will increase CPU cycles for most customers`