Fury0508 commented on PR #60973:
URL: https://github.com/apache/airflow/pull/60973#issuecomment-3790339837
> Thanks for the PR! I just have one question:
>
> It seems user_endpoint.py hasn't been migrated to FastAPI yet, but the
commit message says it has. Did we miss this file?
Thanks for the question!
The new FastAPI endpoints are in `api_fastapi/routes/users.py` (visible in
"Files changed"):
- GET /fab/v1/users
- GET /fab/v1/users/{username}
- PATCH /fab/v1/users/{username}
- DELETE /fab/v1/users/{username}
The old Connexion file is intentionally kept for now. Per issue #44881:
"Endpoints... needs to be converted to FastAPI. (Duplicated for now, legacy
flask endpoint will be removed later)"
This follows the same pattern as PR #58023 (roles migration) - the old
endpoints will be removed in a separate cleanup PR once all sub-issues (#60944,
#60945, #60946) are complete.
--
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]
