Skip to content

Add back methods needed by Flask in AirflowAppBuilder#48968

Merged
vincbeck merged 1 commit into
apache:mainfrom
aws-mwaa:vincbeck/fab_url_login
Apr 9, 2025
Merged

Add back methods needed by Flask in AirflowAppBuilder#48968
vincbeck merged 1 commit into
apache:mainfrom
aws-mwaa:vincbeck/fab_url_login

Conversation

@vincbeck

@vincbeck vincbeck commented Apr 8, 2025

Copy link
Copy Markdown
Contributor

Reported in Slack.

Some methods are not used in Airflow codebase but by Flask itself. We need to add them back.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@vincbeck
vincbeck force-pushed the vincbeck/fab_url_login branch from 609080a to 80e1aed Compare April 8, 2025 18:39
@vincbeck vincbeck changed the title Add back get_url_for_login in AirflowAppBuilder Add back methods needed by Flask in AirflowAppBuilder Apr 8, 2025
@vincbeck
vincbeck force-pushed the vincbeck/fab_url_login branch 2 times, most recently from 9ab6a22 to a0cb8ec Compare April 8, 2025 19:04

@zachliu zachliu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Comment thread providers/fab/src/airflow/providers/fab/auth_manager/security_manager/override.py Outdated
@potiuk

potiuk commented Apr 9, 2025

Copy link
Copy Markdown
Member

With nit from Elad

@vincbeck
vincbeck force-pushed the vincbeck/fab_url_login branch from a0cb8ec to cd12350 Compare April 9, 2025 13:47
@vincbeck
vincbeck requested a review from eladkal April 9, 2025 14:21
@vincbeck
vincbeck merged commit 55c29a0 into apache:main Apr 9, 2025
@vincbeck
vincbeck deleted the vincbeck/fab_url_login branch April 9, 2025 19:18
@vincbeck vincbeck mentioned this pull request Apr 10, 2025
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.

5 participants