-
Notifications
You must be signed in to change notification settings - Fork 14.2k
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
[AIRFLOW-5458] Bump Flask-AppBuilder to 2.2.0 #6607
Conversation
Same fix as was required for Superset (apache/superset#7739)
All in favour of this change, but have you checked the rest of the UI? Has anything else around filtering or searching broken or changed? |
Codecov Report
@@ Coverage Diff @@
## master #6607 +/- ##
=========================================
- Coverage 83.83% 83.5% -0.33%
=========================================
Files 651 651
Lines 37431 37431
=========================================
- Hits 31379 31256 -123
- Misses 6052 6175 +123
Continue to review full report at Codecov.
|
I logged in with two different accounts, one with |
how about a custom DAG user role? |
cc @mistercrunch Max, do you know if FAB changes any API/interface that Airflow is using? |
We've been experiencing a similar issue as AIRFLOW-5462. Everything works as expected for a valid login using the company domain specified in |
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.
I cherry-picked this line to v1-10-test ( 5c3813b )
Can confirm that UI works with Flask-AppBuilder 2.2.1
I'm still having the same error using |
@zachliu Yeah, it sounds like this change won't fix the OAuth issue :( , but will improve a few other things :) |
@paulvic Can you update the PR based on the suggestion and we can use the same PR to resolve https://issues.apache.org/jira/browse/AIRFLOW-5458, we want to do this for 1.10.7 |
This might also fix AIRFLOW-5462 (OAuth login issue) Same fix as was required for Superset (apache/superset#7739) (cherry picked from commit 47facb4)
Fab 2.0.0 only works on py3.6+ - on the release branch this change has been changed so it's this:
|
This might also fix AIRFLOW-5462 (OAuth login issue) Same fix as was required for Superset (apache/superset#7739) (cherry picked from commit 47facb4)
This might also fix AIRFLOW-5462 (OAuth login issue) Same fix as was required for Superset (apache/superset#7739) (cherry picked from commit 47facb4)
This might also fix AIRFLOW-5462 (OAuth login issue) Same fix as was required for Superset (apache/superset#7739) (cherry picked from commit 47facb4)
This might also fix AIRFLOW-5462 (OAuth login issue) Same fix as was required for Superset (apache/superset#7739)
This might also fix AIRFLOW-5462 (OAuth login issue) Same fix as was required for Superset (apache/superset#7739)
Same fix as was required for Superset (apache/superset#7739)
Make sure you have checked all steps below.
Jira
Description
When using OAuth for authentication, if the
next
request parameter isn't set in the URL then the initial redirect is broken after logging in. This was fixed in FAB (Fix, OAuth - set fallback when next url in state is empty dpgaspar/Flask-AppBuilder#1095) and so the version of FAB needs to be bumped to pick up the fix.Tests
I'm not sure if we would add tests for an external package.
I have tested this manually and confirmed the issue is resolved.
Commits
Documentation