relax validation for openapi responses

giograno · giograno · commit a3a42dd8dfdc · 2024-09-09T10:04:57.000-04:00
diff --git a/localstack-core/localstack/aws/handlers/validation.py b/localstack-core/localstack/aws/handlers/validation.py
@@ -12,6 +12,7 @@
 from openapi_core.validation.request.exceptions import (
     RequestValidationError,
 )
+from openapi_core.validation.response.exceptions import ResponseValidationError
 
 from localstack import config
 from localstack.aws.api import RequestContext
@@ -67,7 +68,8 @@ def __call__(self, chain: HandlerChain, context: RequestContext, response: Respo
 
 class OpenAPIResponseValidator(OpenAPIValidator):
     def __call__(self, chain: HandlerChain, context: RequestContext, response: Response):
-        # We are more lenient in validating the responses. The use of this flag is intended for test.
+        # The use of this flag is intended for test only. Eventual errors are due to LocalStack implementation and not
+        #   to improper user usage of the endpoints.
         if not config.OPENAPI_VALIDATE_RESPONSE:
             return
 
@@ -79,7 +81,7 @@ def __call__(self, chain: HandlerChain, context: RequestContext, response: Respo
                     WerkzeugOpenAPIRequest(context.request),
                     WerkzeugOpenAPIResponse(response),
                 )
-            except OpenAPIError as exc:
+            except ResponseValidationError as exc:
                 LOG.error("Response validation failed for %s: $s", path, exc)
                 response.status_code = 500
                 response.set_json({"error": exc.__class__.__name__, "message": str(exc)})
