Skip to content
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.

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

Fix: JWT token auth in Airflow 3 beta as JWT mechanism changed #47835

Merged
merged 1 commit into from
Mar 17, 2025

Conversation

jscheffl
Copy link
Contributor

Unfortunately I needed to notice that the recent changes in JWT token generation/validation broke the Edge Worker API calls in main. Just a small adjustment needed which even reduces a diff between 2.10 and 3.0 line.

This also (again) triggers me that I need to find time to add a Edge Worker test into CI...

How to test?

  • Start via breeze start-airflow --python 3.12 --load-example-dags --backend postgres --executor EdgeExecutor --answer y and see that EdgeWorker comes-up and reports no task. Before this PR on main it failed authentication and got a 403 error.

@boring-cyborg boring-cyborg bot added area:providers provider:edge Edge Executor / Worker (AIP-69) labels Mar 16, 2025
@jscheffl jscheffl requested review from ashb, potiuk and kaxil March 16, 2025 22:02
@jscheffl jscheffl merged commit 59555b4 into apache:main Mar 17, 2025
62 checks passed
tyrellcurry pushed a commit to tyrellcurry/airflow that referenced this pull request Mar 21, 2025
agupta01 pushed a commit to agupta01/airflow that referenced this pull request Mar 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:providers provider:edge Edge Executor / Worker (AIP-69)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants