potiuk commented on code in PR #50839:
URL: https://github.com/apache/airflow/pull/50839#discussion_r2100186220
##########
providers/fab/src/airflow/providers/fab/www/app.py:
##########
@@ -56,7 +56,7 @@ def create_app(enable_plugins: bool):
from airflow.providers.fab.auth_manager.fab_auth_manager import
FabAuthManager
flask_app = Flask(__name__)
- flask_app.secret_key = conf.get("webserver", "SECRET_KEY")
+ flask_app.secret_key = conf.get("api", "SECRET_KEY")
Review Comment:
> I understand. The idea is that since we are in super early 3.0.0 release
cycle, no 3.1.0 yet, we allow ourselves to be in this gray area. Which isn't
ideal, but pretty convenient at this point.
Yes. That's the idea, it's really because we are bug-fixing issues that we
have not seen before we released 3.0.0 and 3.0.1. And yes it means exactly what
you say @eladkal,but I consider that as one-time exception only and result of
having some left-overs not finished in Airflow 3 - we are cleaning it "now".
Simillary task-sdk is now coupled to Airflow - and we only plan to "free it"
when both Airflow 3 and Task SDK reach `X.1.0` verions.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]