Falcon integration

Author: p1c2u

Diff for: openapi_core/contrib/falcon/handlers.py

@@ -0,0 +1,52 @@
+"""OpenAPI core contrib falcon handlers module"""
+from json import dumps
+
+from falcon.constants import MEDIA_JSON
+from falcon.status_codes import (
+    HTTP_400, HTTP_404, HTTP_405, HTTP_415,
+)
+from openapi_core.schema.media_types.exceptions import InvalidContentType
+from openapi_core.templating.paths.exceptions import (
+    ServerNotFound, OperationNotFound, PathNotFound,
+)
+
+
+class FalconOpenAPIErrorsHandler(object):
+
+    OPENAPI_ERROR_STATUS = {
+        ServerNotFound: 400,
+        OperationNotFound: 405,
+        PathNotFound: 404,
+        InvalidContentType: 415,
+    }
+
+    FALCON_STATUS_CODES = {
+        400: HTTP_400,
+        404: HTTP_404,
+        405: HTTP_405,
+        415: HTTP_415,
+    }
+
+    @classmethod
+    def handle(cls, req, resp, errors):
+        data_errors = [
+            cls.format_openapi_error(err)
+            for err in errors
+        ]
+        data = {
+            'errors': data_errors,
+        }
+        data_error_max = max(data_errors, key=lambda x: x['status'])
+        resp.content_type = MEDIA_JSON
+        resp.status = cls.FALCON_STATUS_CODES.get(
+            data_error_max['status'], HTTP_400)
+        resp.body = dumps(data)
+        resp.complete = True
+
+    @classmethod
+    def format_openapi_error(cls, error):
+        return {
+            'title': str(error),
+            'status': cls.OPENAPI_ERROR_STATUS.get(error.__class__, 400),
+            'class': str(type(error)),
+        }

Diff for: openapi_core/contrib/falcon/middlewares.py

@@ -0,0 +1,73 @@
+"""OpenAPI core contrib falcon middlewares module"""
+
+from openapi_core.contrib.falcon.handlers import FalconOpenAPIErrorsHandler
+from openapi_core.contrib.falcon.requests import FalconOpenAPIRequestFactory
+from openapi_core.contrib.falcon.responses import FalconOpenAPIResponseFactory
+from openapi_core.validation.processors import OpenAPIProcessor
+from openapi_core.validation.request.validators import RequestValidator
+from openapi_core.validation.response.validators import ResponseValidator
+
+
+class FalconOpenAPIMiddleware(OpenAPIProcessor):
+
+    def __init__(
+            self,
+            request_validator,
+            response_validator,
+            request_factory,
+            response_factory,
+            openapi_errors_handler,
+    ):
+        super(FalconOpenAPIMiddleware, self).__init__(
+            request_validator, response_validator)
+        self.request_factory = request_factory
+        self.response_factory = response_factory
+        self.openapi_errors_handler = openapi_errors_handler
+
+    def process_request(self, req, resp):
+        openapi_req = self._get_openapi_request(req)
+        req_result = super(FalconOpenAPIMiddleware, self).process_request(
+            openapi_req)
+        if req_result.errors:
+            return self._handle_request_errors(req, resp, req_result)
+        req.openapi = req_result
+
+    def process_response(self, req, resp, resource, req_succeeded):
+        openapi_req = self._get_openapi_request(req)
+        openapi_resp = self._get_openapi_response(resp)
+        resp_result = super(FalconOpenAPIMiddleware, self).process_response(
+            openapi_req, openapi_resp)
+        if resp_result.errors:
+            return self._handle_response_errors(req, resp, resp_result)
+
+    def _handle_request_errors(self, req, resp, request_result):
+        return self.openapi_errors_handler.handle(
+            req, resp, request_result.errors)
+
+    def _handle_response_errors(self, req, resp, response_result):
+        return self.openapi_errors_handler.handle(
+            req, resp, response_result.errors)
+
+    def _get_openapi_request(self, request):
+        return self.request_factory.create(request)
+
+    def _get_openapi_response(self, response):
+        return self.response_factory.create(response)
+
+    @classmethod
+    def from_spec(
+            cls,
+            spec,
+            request_factory=FalconOpenAPIRequestFactory,
+            response_factory=FalconOpenAPIResponseFactory,
+            openapi_errors_handler=FalconOpenAPIErrorsHandler,
+    ):
+        request_validator = RequestValidator(spec)
+        response_validator = ResponseValidator(spec)
+        return cls(
+            request_validator=request_validator,
+            response_validator=response_validator,
+            request_factory=request_factory,
+            response_factory=response_factory,
+            openapi_errors_handler=openapi_errors_handler,
+        )

Diff for: openapi_core/contrib/falcon/requests.py

@@ -1,36 +1,45 @@
 """OpenAPI core contrib falcon responses module"""
-import json
+from json import dumps
 
-from openapi_core.validation.request.datatypes import OpenAPIRequest, RequestParameters
 from werkzeug.datastructures import ImmutableMultiDict
 
+from openapi_core.validation.request.datatypes import (
+    OpenAPIRequest, RequestParameters,
+)
+
 
 class FalconOpenAPIRequestFactory:
+
     @classmethod
-    def create(cls, req, route_params):
+    def create(cls, request):
         """
         Create OpenAPIRequest from falcon Request and route params.
         """
-        method = req.method.lower()
+        method = request.method.lower()
 
-        # Convert keys to lowercase as that's what the OpenAPIRequest expects.
-        headers = {key.lower(): value for key, value in req.headers.items()}
+        # gets deduced by path finder against spec
+        path = {}
+
+        # Support falcon-jsonify.
+        body = (
+            dumps(request.json) if getattr(request, "json", None)
+            else request.bounded_stream.read()
+        )
+        mimetype = request.options.default_media_type
+        if request.content_type:
+            mimetype = request.content_type.partition(";")[0]
 
+        query = ImmutableMultiDict(request.params.items())
         parameters = RequestParameters(
-            path=route_params,
-            query=ImmutableMultiDict(req.params.items()),
-            header=headers,
-            cookie=req.cookies,
+            query=query,
+            header=request.headers,
+            cookie=request.cookies,
+            path=path,
         )
         return OpenAPIRequest(
-            host_url=None,
-            path=req.path,
-            path_pattern=req.uri_template or req.path,
+            full_url_pattern=request.url,
             method=method,
             parameters=parameters,
-            # Support falcon-jsonify.
-            body=json.dumps(req.json)
-            if getattr(req, "json", None)
-            else req.bounded_stream.read(),
-            mimetype=req.content_type.partition(";")[0] if req.content_type else "",
+            body=body,
+            mimetype=mimetype,
         )

Diff for: openapi_core/contrib/falcon/responses.py

@@ -4,9 +4,17 @@
 
 class FalconOpenAPIResponseFactory(object):
     @classmethod
-    def create(cls, resp):
+    def create(cls, response):
+        status_code = int(response.status[:3])
+
+        mimetype = ''
+        if response.content_type:
+            mimetype = response.content_type.partition(";")[0]
+        else:
+            mimetype = response.options.default_media_type
+
         return OpenAPIResponse(
-            data=resp.body,
-            status_code=resp.status[:3],
-            mimetype=resp.content_type.partition(";")[0] if resp.content_type else '',
+            data=response.body,
+            status_code=status_code,
+            mimetype=mimetype,
         )

Diff for: openapi_core/contrib/falcon/views.py

@@ -1,8 +1,7 @@
-from falcon import Request, Response
+from falcon import Request, Response, RequestOptions, ResponseOptions
 from falcon.routing import DefaultRouter
 from falcon.testing import create_environ
 import pytest
-from six import BytesIO
 
 
 @pytest.fixture
@@ -18,20 +17,22 @@ def create_env(method, path, server_name):
 @pytest.fixture
 def router():
     router = DefaultRouter()
-    router.add_route('/browse/<int:id>/', None)
+    router.add_route("/browse/{id:int}/", lambda x: x)
     return router
 
 
 @pytest.fixture
 def request_factory(environ_factory, router):
     server_name = 'localhost'
 
-    def create_request(method, path, subdomain=None, query_string=None):
+    def create_request(
+            method, path, subdomain=None, query_string=None,
+            content_type='application/json'):
         environ = environ_factory(method, path, server_name)
-        options = None
+        options = RequestOptions()
         # return create_req(options=options, **environ)
         req = Request(environ, options)
-        req.uri_template = router.find(path, req)
+        resource, method_map, params, req.uri_template = router.find(path, req)
         return req
     return create_request
 
@@ -40,10 +41,8 @@ def create_request(method, path, subdomain=None, query_string=None):
 def response_factory(environ_factory):
     def create_response(
             data, status_code=200, content_type='application/json'):
-        options = {
-            'content_type': content_type,
-            'data': data,
-            'status': status_code,
-        }
-        return Response(options)
+        options = ResponseOptions()
+        resp = Response(options)
+        resp._media = data
+        return resp
     return create_response

Diff for: tests/integration/contrib/falcon/conftest.py

@@ -5,7 +5,7 @@ info:
 servers:
   - url: 'http://localhost'
 paths:
-  '/browse/{id}/':
+  '/browse/{id}':
     parameters:
       - name: id
         in: path