gaurav0107 commented on code in PR #65685:
URL: https://github.com/apache/airflow/pull/65685#discussion_r3126687546
##########
airflow-core/src/airflow/api_fastapi/core_api/app.py:
##########
@@ -174,6 +175,9 @@ def init_middlewares(app: FastAPI) -> None:
app.add_middleware(SimpleAllAdminMiddleware)
Review Comment:
Done — removed the hardcoded `SimpleAllAdminMiddleware` branch from
`init_middlewares` and pushed the registration into
`SimpleAuthManager.get_fastapi_middlewares`. Both FAB and simple now go through
the same generic hook. Added unit tests for the new override too.
--
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]
