FABAuthManager migrate endpoints to FastAPI #44881
Assignees
Labels
area:API
Airflow's REST/HTTP API
area:auth
good first issue
kind:feature
Feature Requests
provider:fab
Comments
pierrejeambrun
added
kind:feature
|
cc: @vincbeck |
1 task
|
Hi @pierrejeambrun, |
|
Assigned to you @andreimek , thanks |
|
Related to the discussion in #44883, we likely don't need to actually rewrite or convert these, just wrap them with WSGIMiddleware when auth managers start exposing fastapi subapps. |
|
Hey @andreimek , did you get a chance to work on this issue? Please do let us know if you need any help. |
|
Should we close this one? My understanding is we will not migrate the endpoints but just embed the Flask endpoint into the Fastapi? |
|
Yeah, and #45009 is already doing that mounting. |
github-project-automation
bot
moved this from Todo
to Done
in AIP-79: Remove Flask AppBuilder as Core dependency
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Description
Depends on: #44847
FABAuthManager defines extra flask endpoints.
As part of the migration to FastAPI these endpoints need to be converted to FastAPI.
Endpoints located in
airflow/providers/fab/auth_manager/api_endpointsneeds to be converted to FastAPI. (Duplicated for now, legacy flask endpoint will be removed later)The text was updated successfully, but these errors were encountered: