Skip to content

[WIP] Add FastAPI Depends for Auth Manager #50585

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 1 commit into
base: main
Choose a base branch
from
Draft

[WIP] Add FastAPI Depends for Auth Manager #50585

wants to merge 1 commit into from

Conversation

jason810496
Copy link
Member

@jason810496 jason810496 commented May 14, 2025 •
edited
Loading

related: #50372
related comment: #50372 (comment)

Why

Follow-up PR for Replace DagBag from global app state to with FastAPI dependency #50372, we should also add FastAPI Depends for Auth Manger.

What

  • Move initialization of Auth Manager from airflow.api_fastapi.app to airflow.api_fastapi.common.auth_manager module.
  • Add FastAPI depends and also ensure singleton for Auth Manager.

@github-actions GitHub Actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity. It will be closed in 5 days if no further activity occurs. Thank you for your contributions.

@github-actions github-actions bot added the stale Stale PRs per the .github/workflows/stale.yml policy file label Jun 29, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@vincbeck vincbeck Awaiting requested review from vincbeck vincbeck will be requested when the pull request is marked ready for review vincbeck is a code owner

@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy will be requested when the pull request is marked ready for review ephraimbuddy is a code owner

@pierrejeambrun pierrejeambrun Awaiting requested review from pierrejeambrun pierrejeambrun will be requested when the pull request is marked ready for review pierrejeambrun is a code owner

@rawwar rawwar Awaiting requested review from rawwar rawwar will be requested when the pull request is marked ready for review rawwar is a code owner

@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 will be requested when the pull request is marked ready for review bugraoz93 is a code owner

@shubhamraj-git shubhamraj-git Awaiting requested review from shubhamraj-git shubhamraj-git will be requested when the pull request is marked ready for review shubhamraj-git is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
area:API Airflow's REST/HTTP API area:CLI area:providers provider:fab stale Stale PRs per the .github/workflows/stale.yml policy file
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jason810496