abhijeets25012-tech commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2747252107
##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -185,3 +183,5 @@ def init_middlewares(app: FastAPI) -> None:
from airflow.api_fastapi.auth.managers.simple.middleware import
SimpleAllAdminMiddleware
app.add_middleware(SimpleAllAdminMiddleware)
+
+ app.add_middleware(GZipMiddleware, minimum_size=1024, compresslevel=5)
Review Comment:
Thanks for the review and for the feedback.
Good point on the comment — I’ll move the GZipMiddleware-related comment
next to the middleware itself to keep things consistent.
Regarding testing: yes, I did verify the header behavior locally using httpx
after adjusting the middleware order. I agree that this needs to be protected
by a test to avoid regressions. I’ll add a test case asserting that
Transfer-Encoding: chunked is not reintroduced.
I’ll push an updated commit shortly with the comment fix and the test.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
