-
Notifications
You must be signed in to change notification settings - Fork 16.6k
Handle invalid token in JWTRefreshMiddleware#56904
Conversation
When the token is invalid, the middleware JWTRefreshMiddleware creates a 500. Handle this use case. An easy reproducible step is to not set AIRFLOW__API_AUTH__JWT_SECRET, and restarting breeze while keeping your browser open.
See discussion in Slack.
^ 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.
|
The PR that broke it... did this go to v3-1-test and do we need to backport this? |
jscheffl
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can confirmt his fixes it! Thanks for helping out (faster than I was able to make my hands dirty...)
No, |
Apparently that changed, backport PR #58649, so we need to backport this too. |
|
@vincbeck Ephraim is on it. (same for the other PR). As he is cutting rc2 |