You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "abhijeets25012-tech (via GitHub)" <gi...@apache.org> on 2026/01/25 17:25:40 UTC

[PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

abhijeets25012-tech opened a new pull request, #61043:
URL: https://github.com/apache/airflow/pull/61043

   
   Fixes #<issue-number>
   
   Reorders JWTRefreshMiddleware, SimpleAllAdminMiddleware, and GZipMiddleware
   to restore non-chunked API responses and prevent nginx duplicate Transfer-Encoding headers.
   
   - JWTRefreshMiddleware runs first
   - Optional SimpleAllAdminMiddleware runs second
   - GZipMiddleware runs last
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "SameerMesiah97 (via GitHub)" <gi...@apache.org>.
SameerMesiah97 commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3797271647

   Can you fix the issue link?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3861085177

   > Got it. Thanks for the reminder! I will double-check the milestone before merging next time.
   
   And do the backport. (people merging are responsible for backporting too if I recall correctly :p) No worries it's easy to forget.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3818832918

   Ok , I will fix it


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "jason810496 (via GitHub)" <gi...@apache.org>.
jason810496 merged PR #61043:
URL: https://github.com/apache/airflow/pull/61043


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3861068570

   @jason810496 please make sure to at least at the milestone when merging the PR and ideally do the backport, otherwise this will not be get released in the next patch release and this fix will wait until next minor.
   
   Manual backport:
   https://github.com/apache/airflow/pull/61539


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "jason810496 (via GitHub)" <gi...@apache.org>.
jason810496 commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2741147880


##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -185,3 +183,4 @@ 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:
   It would be nice to move the comment of GZipMiddleware down here as well, and we could remove the existed comments above. 
   
   ```suggestion
       # Compress responses greater than 1kB with optimal compression level as 5
       # with level ranging from 1 to 9 with 1 (fastest, least compression)
       # and 9 (slowest, most compression)
       app.add_middleware(GZipMiddleware, minimum_size=1024, compresslevel=5)
   ```
   
   Thanks.



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2747035395


##########
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:
   You need to move the comment that was related to this middleware in the original code.



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "vincbeck (via GitHub)" <gi...@apache.org>.
vincbeck commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3818581620

   Statc checks are still failing. Please read the [documentation](https://github.com/apache/airflow/blob/main/contributing-docs/08_static_code_checks.rst) to learn more about how to fix them.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "boring-cyborg[bot] (via GitHub)" <gi...@apache.org>.
boring-cyborg[bot] commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3845725247

   Awesome work, congrats on your first merged pull request! You are invited to check our [Issue Tracker](https://github.com/apache/airflow/issues) for additional contributions.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "vincbeck (via GitHub)" <gi...@apache.org>.
vincbeck commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3812817979

   Agree with the e2e but the static checks are on your side ;) Running `prek --files airflow-core/src/airflow/api_fastapi/core_api/app.py b/airflow-core/src/airflow/api_fastapi/core_api/app.py` should solve the issue


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2727105606


##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -180,8 +181,11 @@ def init_error_handlers(app: FastAPI) -> None:
 def init_middlewares(app: FastAPI) -> None:
     from airflow.api_fastapi.auth.middlewares.refresh_token import JWTRefreshMiddleware
 
+    
     app.add_middleware(JWTRefreshMiddleware)
     if conf.getboolean("core", "simple_auth_manager_all_admins"):
         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:
   Fix middleware order in FastAPI app
   
   - Added JWTRefreshMiddleware first
   - Added SimpleAllAdminMiddleware if enabled
   - GZipMiddleware runs last
   - Ensures proper API response and avoids chunked transfer issues
   



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3841537606

   Thanks for fixing the CI and syncing the branch 👍
   I’ll follow the #61338 contribution guideline for future changes. Please let me know if anything else is needed from my side.
   
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "jason810496 (via GitHub)" <gi...@apache.org>.
jason810496 commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3861080593

   > please make sure to at least add the milestone when merging the PR and ideally do the backport, otherwise this will not get released in the next patch release and this fix will wait until next minor. (if both backport is not done and milestone is not set)
   
   Got it. Thanks for the reminder! I will double-check the milestone before merging next time.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2736777294


##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -165,7 +165,8 @@ def init_config(app: FastAPI) -> None:
     # Compress responses greater than 1kB with optimal compression level as 5
     # with level ranging from 1 to 9 with 1 (fastest, least compression)
     # and 9 (slowest, most compression)
-    app.add_middleware(GZipMiddleware, minimum_size=1024, compresslevel=5)
+    
+   

Review Comment:
   Ok thanks for your guidence



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3812634726

   Thanks for the review!
   
   All code-related checks are green. The remaining failures seem to be CI-related:
   - Static checks (CI tooling/environment)
   - Firefox UI e2e test (appears flaky)
   
   Requesting a re-run of the failing jobs.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "pierrejeambrun (via GitHub)" <gi...@apache.org>.
pierrejeambrun commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2762779731


##########
airflow-core/tests/unit/api_fastapi/core_api/test_app.py:
##########
@@ -0,0 +1,38 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+from __future__ import annotations
+
+import pytest
+
+from tests_common.test_utils.db import clear_db_jobs
+
+pytestmark = pytest.mark.db_test
+
+
+class TestGzipMiddleware:
+    @pytest.fixture(autouse=True)
+    def setup(self):

Review Comment:
   Not sure why we need that setup function.



##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -185,3 +180,9 @@ def init_middlewares(app: FastAPI) -> None:
         from airflow.api_fastapi.auth.managers.simple.middleware import SimpleAllAdminMiddleware
 
         app.add_middleware(SimpleAllAdminMiddleware)
+
+    # The GzipMiddleware should be the last middleware added as https://github.com/apache/airflow/issues/60165 points out.
+    app.add_middleware(GZipMiddleware, minimum_size=1024, compresslevel=5)
+    # Compress responses greater than 1kB with optimal compression level as 5
+    # with level ranging from 1 to 9 with 1 (fastest, least compression)
+    # and 9 (slowest, most compression)

Review Comment:
   This comment is probably missplaced and should be above the GZipMiddleware line.



##########
airflow-core/tests/unit/api_fastapi/core_api/test_app.py:
##########
@@ -0,0 +1,38 @@
+# Licensed to the Apache Software Foundation (ASF) under one
+# or more contributor license agreements.  See the NOTICE file
+# distributed with this work for additional information
+# regarding copyright ownership.  The ASF licenses this file
+# to you under the Apache License, Version 2.0 (the
+# "License"); you may not use this file except in compliance
+# with the License.  You may obtain a copy of the License at
+#
+#   http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing,
+# software distributed under the License is distributed on an
+# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
+# KIND, either express or implied.  See the License for the
+# specific language governing permissions and limitations
+# under the License.
+from __future__ import annotations
+
+import pytest
+
+from tests_common.test_utils.db import clear_db_jobs
+
+pytestmark = pytest.mark.db_test

Review Comment:
   Not sure why it's marked as a db 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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2747395796


##########
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:
   Added an integration test to ensure the health endpoint does not return Transfer-Encoding: chunked, preventing regression due to middleware ordering.



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "Srabasti (via GitHub)" <gi...@apache.org>.
Srabasti commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2735013703


##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -165,7 +165,8 @@ def init_config(app: FastAPI) -> None:
     # Compress responses greater than 1kB with optimal compression level as 5
     # with level ranging from 1 to 9 with 1 (fastest, least compression)
     # and 9 (slowest, most compression)
-    app.add_middleware(GZipMiddleware, minimum_size=1024, compresslevel=5)
+    
+   

Review Comment:
   Looks like extra newlines added are causing static checks to fail @abhijeets25012-tech
   
   Can you run prek locally first and then push the changes and then push commit from your branch?
   Prek fixes any formatting errors, automatically.



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "jason810496 (via GitHub)" <gi...@apache.org>.
jason810496 commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2726259180


##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -180,8 +181,11 @@ def init_error_handlers(app: FastAPI) -> None:
 def init_middlewares(app: FastAPI) -> None:
     from airflow.api_fastapi.auth.middlewares.refresh_token import JWTRefreshMiddleware
 
+    
     app.add_middleware(JWTRefreshMiddleware)
     if conf.getboolean("core", "simple_auth_manager_all_admins"):
         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:
   Could we move the comments as well?



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on code in PR #61043:
URL: https://github.com/apache/airflow/pull/61043#discussion_r2741297654


##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -185,3 +183,4 @@ 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:
   fixed 



-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3817215287

   I fixed the EOF newline and spacing issue in app.py.
   The middleware block is now formatted correctly.
   Local prek cannot be fully executed without Docker/CI tooling, but the formatting issue it checks is resolved.
   Requesting a re-run of the static checks.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "abhijeets25012-tech (via GitHub)" <gi...@apache.org>.
abhijeets25012-tech commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3841189149

   Thanks for updating the comments and adjusting the test convention.
   
   I’ve synced my branch locally and everything looks good from my side. Please let me know if any further changes or validation are needed.
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] Fix middleware order to prevent chunked FastAPI responses [airflow]

Posted by "jason810496 (via GitHub)" <gi...@apache.org>.
jason810496 commented on PR #61043:
URL: https://github.com/apache/airflow/pull/61043#issuecomment-3841312999

   Bad rebase, sorry for tagging everyone.


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org