Skip to content

Fix asset state store writes forbidden under FAB auth manager - #70478

Merged
vincbeck merged 1 commit into
apache:mainfrom
justinpakzad:fix/fab-asset-state-store-permission
Jul 28, 2026
Merged

Fix asset state store writes forbidden under FAB auth manager#70478
vincbeck merged 1 commit into
apache:mainfrom
justinpakzad:fix/fab-asset-state-store-permission

Conversation

@justinpakzad

@justinpakzad justinpakzad commented Jul 26, 2026

Copy link
Copy Markdown
Contributor

FAB auth manager never registered (can_edit, Asset), so PUT /assets/{asset_id}/state-store/{key} returns 403 for every user including admin, leaving asset state store writes via the REST API unreachable on FAB deployments. This PR adds the permission to OP_PERMISSIONS.

Was generative AI tooling used to co-author this PR?
  • Yes (Claude Opus 4.6)

  • 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, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@justinpakzad
justinpakzad marked this pull request as ready for review July 26, 2026 22:08
@justinpakzad
justinpakzad requested a review from vincbeck as a code owner July 26, 2026 22:08
Comment thread providers/fab/tests/unit/fab/auth_manager/test_security.py Outdated
@justinpakzad
justinpakzad force-pushed the fix/fab-asset-state-store-permission branch from 2ce2099 to 8f3f81e Compare July 27, 2026 20:59
@vincbeck
vincbeck merged commit 6575b3f into apache:main Jul 28, 2026
79 checks passed
@potiuk potiuk added the ready for maintainer review Set after triaging when all criteria pass. label Jul 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:fab ready for maintainer review Set after triaging when all criteria pass.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants