Skip to content

Deprecated access method removed - #41738

Closed
dirrao wants to merge 5 commits into
apache:mainfrom
dirrao:deprecated_www_has_access_method_removal
Closed

Deprecated access method removed#41738
dirrao wants to merge 5 commits into
apache:mainfrom
dirrao:deprecated_www_has_access_method_removal

Conversation

@dirrao

@dirraodirrao commented Aug 26, 2024•
edited
Loading

Copy link
Copy Markdown
Contributor

Removed the following depredated methods

  1. Removed deprecated method has_access from airflow.www.auth
  2. Removed deprecated method requires_access from airflow.api_connexion.security
  3. Removed deprecated methods get_sensitive_variables_fields and should_hide_value_for_key from airflow.www.utils.

@dirraodirrao self-assigned this Aug 26, 2024
@dirraodirrao added the airflow3.0:candidate Potential candidates for Airflow 3.0 label Aug 26, 2024
@dirraodirrao added the airflow3.0:breaking Candidates for Airflow 3.0 that contain breaking changes label Aug 26, 2024
@dirrao
dirrao requested a review from jhtimmins as a code ownerAugust 26, 2024 05:43
@dirraodirrao changed the title Deprecated airflow.www.auth.has_access method removedDeprecated access method removedAug 26, 2024
eladkal
eladkal reviewed Aug 26, 2024
View reviewed changes
Comment threadairflow/providers/databricks/plugins/databricks_workflow.py Outdated
@dirraodirrao mentioned this pull request Aug 26, 2024
Merged
@jscheffl

jscheffl commented Aug 26, 2024

Copy link
Copy Markdown
Contributor

Oh and same here - did the same in parallel :-D in PR #41758

@dirrao
dirrao requested review from eladkal and vincbeckAugust 27, 2024 04:25
@potiuk

potiuk commented Aug 27, 2024

Copy link
Copy Markdown
Member

Already merged in #41758 this time

@potiukpotiuk closed this Aug 27, 2024
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ryanahamiltonryanahamiltonAwaiting requested review from ryanahamilton
@ashbashbAwaiting requested review from ashb
@bbovenzibbovenziAwaiting requested review from bbovenzi
@pierrejeambrunpierrejeambrunAwaiting requested review from pierrejeambrun
@jhtimminsjhtimminsAwaiting requested review from jhtimmins
@eladkaleladkalAwaiting requested review from eladkal
@vincbeckvincbeckAwaiting requested review from vincbeck

Assignees

@dirraodirrao

Labels

airflow3.0:breakingCandidates for Airflow 3.0 that contain breaking changesairflow3.0:candidatePotential candidates for Airflow 3.0area:providersarea:webserverWebserver related Issuesprovider:databricks

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@dirrao@jscheffl@potiuk@eladkal@vincbeck