jason810496 commented on code in PR #53567:
URL: https://github.com/apache/airflow/pull/53567#discussion_r2220894558
##########
airflow-core/src/airflow/api_fastapi/logging/decorators.py:
##########
@@ -22,17 +22,19 @@
from typing import Annotated
import pendulum
-from fastapi import Depends, Request
+from fastapi import Depends, Request, Security
from pendulum.parsing.exceptions import ParserError
from airflow.api_fastapi.auth.managers.models.base_user import BaseUser
from airflow.api_fastapi.common.db.common import SessionDep
-from airflow.api_fastapi.core_api.security import
get_user_with_exception_handling
+from airflow.api_fastapi.core_api.security import get_user
Review Comment:
```suggestion
from airflow.api_fastapi.core_api.security import GetUserDep
```
##########
airflow-core/src/airflow/api_fastapi/logging/decorators.py:
##########
@@ -76,7 +78,7 @@ def action_logging(event: str | None = None):
async def log_action(
request: Request,
session: SessionDep,
- user: Annotated[BaseUser, Depends(get_user_with_exception_handling)],
+ user: BaseUser = Security(get_user),
Review Comment:
```suggestion
user: GetUserDep,
```
##########
airflow-core/src/airflow/api_fastapi/logging/decorators.py:
##########
@@ -22,17 +22,19 @@
from typing import Annotated
import pendulum
-from fastapi import Depends, Request
+from fastapi import Depends, Request, Security
from pendulum.parsing.exceptions import ParserError
from airflow.api_fastapi.auth.managers.models.base_user import BaseUser
from airflow.api_fastapi.common.db.common import SessionDep
-from airflow.api_fastapi.core_api.security import
get_user_with_exception_handling
+from airflow.api_fastapi.core_api.security import get_user
from airflow.models import Log
from airflow.sdk.execution_time import secrets_masker
logger = logging.getLogger(__name__)
+GetUserDep = Annotated[BaseUser, Depends(get_user)]
Review Comment:
```suggestion
```
--
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]
