improv: add logging; linting

Author: heitorlessa

aws_lambda_powertools/utilities/validation/base.py

@@ -1,10 +1,13 @@
 import json
+import logging
 from typing import Dict
 
 import fastjsonschema
 
 from .exceptions import InvalidEnvelopeExpressionError, InvalidSchemaError, SchemaValidationError
 
+logger = logging.getLogger(__name__)
+
 try:
     import jmespath
 
@@ -36,6 +39,7 @@ def unwrap_event_from_envelope(data: Dict, envelope: str, jmespath_options: Dict
         jmespath_options = {"custom_functions": PowertoolsJson()}
 
     try:
+        logger.debug(f"Envelope detected: {envelope}. JMESPath options: {jmespath_options}")
         return jmespath.search(envelope, data, options=jmespath.Options(**jmespath_options))
     except jmespath.exceptions.LexerError as e:
         raise InvalidEnvelopeExpressionError(e)

aws_lambda_powertools/utilities/validation/validator.py

@@ -1,4 +1,3 @@
-import json
 import logging
 from typing import Any, Callable, Dict, Union
 
@@ -18,9 +17,6 @@ def validator(
     envelope: str = None,
     jmespath_options: Dict = None,
 ):
-    if isinstance(event, str):
-        event = json.loads(event)
-
     if envelope:
         event = unwrap_event_from_envelope(data=event, envelope=envelope, jmespath_options=jmespath_options)
 
@@ -41,7 +37,4 @@ def validate(event: Dict, schema: Dict = None, envelope: str = None, jmespath_op
     if envelope:
         event = unwrap_event_from_envelope(data=event, envelope=envelope, jmespath_options=jmespath_options)
 
-    if isinstance(event, str):
-        event = json.loads(event)
-
     validate_data_against_schema(data=event, schema=schema)