Comment out middleware temporarily

Author: erosselli

src/fides/api/app_setup.py

@@ -93,24 +93,24 @@ def create_fides_app(
     fastapi_app.state.limiter = fides_limiter
     # Starlette bug causing this to fail mypy
     fastapi_app.add_exception_handler(RateLimitExceeded, _rate_limit_exceeded_handler)  # type: ignore
-    fastapi_app.add_middleware(SecurityHeadersMiddleware)
+    # fastapi_app.add_middleware(SecurityHeadersMiddleware)
     for handler in ExceptionHandlers.get_handlers():
         # Starlette bug causing this to fail mypy
         fastapi_app.add_exception_handler(RedisNotConfigured, handler)  # type: ignore
 
-    if is_rate_limit_enabled:
-        # Validate header before SlowAPI processes the request
-        fastapi_app.add_middleware(RateLimitIPValidationMiddleware)
-        # Required for default rate limiting to work
-        fastapi_app.add_middleware(SlowAPIMiddleware)
-    else:
-        logger.warning(
-            "Rate limiting client IPs is disabled because the FIDES__SECURITY__RATE_LIMIT_CLIENT_IP_HEADER env var is not configured."
-        )
-
-    fastapi_app.add_middleware(
-        GZipMiddleware, minimum_size=1000, compresslevel=5
-    )  # minimum_size is in bytes
+    # if is_rate_limit_enabled:
+    #     # Validate header before SlowAPI processes the request
+    #     fastapi_app.add_middleware(RateLimitIPValidationMiddleware)
+    #     # Required for default rate limiting to work
+    #     fastapi_app.add_middleware(SlowAPIMiddleware)
+    # else:
+    #     logger.warning(
+    #         "Rate limiting client IPs is disabled because the FIDES__SECURITY__RATE_LIMIT_CLIENT_IP_HEADER env var is not configured."
+    #     )
+
+    # fastapi_app.add_middleware(
+    #     GZipMiddleware, minimum_size=1000, compresslevel=5
+    # )  # minimum_size is in bytes
 
     for router in routers:
         fastapi_app.include_router(router)

src/fides/api/main.py

@@ -135,8 +135,7 @@ async def lifespan(wrapped_app: FastAPI) -> AsyncGenerator[None, None]:
 
 
 if CONFIG.dev_mode:
-
-    @app.middleware("http")
+    # @app.middleware("http")
     async def profile_request(request: Request, call_next: Callable) -> Response:
         profiling = request.headers.get("profile-request", False)
         if profiling:
@@ -150,7 +149,7 @@ async def profile_request(request: Request, call_next: Callable) -> Response:
         return await call_next(request)
 
 
-@app.middleware("http")
+# @app.middleware("http")
 async def dispatch_log_request(request: Request, call_next: Callable) -> Response:
     """
     HTTP Middleware that logs analytics events for each call to Fides endpoints.
@@ -229,7 +228,7 @@ async def prepare_and_log_request(
     )
 
 
-@app.middleware("http")
+# @app.middleware("http")
 async def log_request(request: Request, call_next: Callable) -> Response:
     """Log basic information about every request handled by the server."""
     start = datetime.now()
@@ -360,7 +359,7 @@ def start_webserver(port: int = 8080) -> None:
     server.run()
 
 
-@app.middleware("http")
+# @app.middleware("http")
 async def action_to_audit_log(
     request: Request,
     call_next: Callable,