-
Notifications
You must be signed in to change notification settings - Fork 16.4k
Add read config permission to viewer role in FabAuthManager
#49581
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
Conversation
bc9b6d2 to
bd96913
Compare
pierrejeambrun
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.
Thanks Vincent
|
When will the 3.0.1 release be released ? |
We are targeting 2 weeks from now. Unless we find a reason to do a patch release earlier. |
|
Am using 3.0.3 and still seeing below error - Have set base url to use my linux server name too Event tried setting export AIRFLOW__API__WEB_SERVER_PORT=8081 10.0.2.100:33622 - "GET / HTTP/1.1" 200 OK Let me know asap. Thanks |
|
Which Airflow version are you using? Because now, you only need to be authenticated to access |
|
Am using 3.0.3. I've created an admin user which I want to use to login to api-server. But on launching it is going in to redirection loop. |
|
What version of fab provider are you using ? Maybe you're behind on that? (also make sure to clear your browser data, for instance try in incognito mode to confirm that's not due to local remnant of cached data) |
|
I noticed am using apache-airflow-providers-fab==2.0.1 with Flask-AppBuilder==4.5.2. Believe airflow 3.0.3 requires - Flask-AppBuilder==4.6.3 & apache-airflow-providers-fab==2.3.0 Can you confirm if this could the root cause? |
I recommend you to use both the latest versions. Some work has been done in both core Airflow and FAB provider so you might have ended up in a very special situation. I am fairly certain that upgrading to both latest version would resolve your issue |
Related #49495.
One of the first thing the UI does is fetching the config. If the user does not have permission to read config, the UI logs out the user. Adding read config as part of
viewerrole inFabAuthManager.^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.