Skip to content

Comments

Migrate Flask based role_and_permission_endpoint APIs to Fastapi#60977

Merged
vincbeck merged 2 commits intoapache:mainfrom
henry3260:replace-fastapi-role-and-permission-endpoint
Feb 2, 2026
Merged

Migrate Flask based role_and_permission_endpoint APIs to Fastapi#60977
vincbeck merged 2 commits intoapache:mainfrom
henry3260:replace-fastapi-role-and-permission-endpoint

Conversation

@henry3260
Copy link
Contributor

@henry3260 henry3260 commented Jan 23, 2026

What's Changed

New endpoints in api_fastapi/routes/roles.py:

GET /fab/v1/permissions - List all permissions

New datamodels in api_fastapi/datamodels/roles.py:

PermissionCollectionResponse - Response model for a paginated list of available permissions in the system.

New service methods in api_fastapi/services/roles.py:

get_permissions() - Queries the DB for Permission rows and returns a PermissionCollectionResponse containing ActionResource items.

closes: #60944

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
    Copilot

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@henry3260 henry3260 requested a review from vincbeck as a code owner January 23, 2026 13:13
@henry3260 henry3260 marked this pull request as draft January 23, 2026 13:40
@henry3260 henry3260 force-pushed the replace-fastapi-role-and-permission-endpoint branch 7 times, most recently from 78ef595 to 72b0fc5 Compare January 24, 2026 01:46
@henry3260 henry3260 force-pushed the replace-fastapi-role-and-permission-endpoint branch 2 times, most recently from baf015f to 90b7ce3 Compare January 24, 2026 12:51
@henry3260 henry3260 force-pushed the replace-fastapi-role-and-permission-endpoint branch from 90b7ce3 to d63770a Compare January 24, 2026 14:03
@henry3260 henry3260 changed the title Replace connexion with fast-api role_and_permission_endpoint Migrate Flask based role_and_permission_endpoint APIs to Fastapi Jan 24, 2026
@henry3260 henry3260 marked this pull request as ready for review January 24, 2026 18:49
@henry3260
Copy link
Contributor Author

cc @vincbeck, Could you review this when you get a chance? Thanks! :)

Copy link
Contributor

@vincbeck vincbeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Good job :)

@vincbeck vincbeck merged commit 8c96236 into apache:main Feb 2, 2026
162 checks passed
@henry3260 henry3260 deleted the replace-fastapi-role-and-permission-endpoint branch February 2, 2026 15:59
jason810496 pushed a commit to abhijeets25012-tech/airflow that referenced this pull request Feb 3, 2026
Alok-kumar-priyadarshi pushed a commit to Alok-kumar-priyadarshi/airflow that referenced this pull request Feb 5, 2026
jhgoebbert pushed a commit to jhgoebbert/airflow_Owen-CH-Leung that referenced this pull request Feb 8, 2026
Ratasa143 pushed a commit to Ratasa143/airflow that referenced this pull request Feb 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace connexion with fast-api: role_and_permission_endpoint.py

2 participants