kalluripradeep commented on PR #64908: URL: https://github.com/apache/airflow/pull/64908#issuecomment-4242619789
Yes, I can say with confidence that this fixes #64151 and provides a more resilient authentication flow. The underlying issue was that certain execution paths (like early FastAPI requests or specific CLI commands) would access the FAB auth routes before the main Airflow init_app phase was completed, leading to an uninitialized AppBuilder and a 500 error. By implementing thread-safe lazy initialization in _get_flask_app(), we ensure that the Flask app context is created on-demand if it hasn't been initialized yet. I've added a regression test in test_login_lazy_init.py that specifically exercises this fallback path, and it has been verified in CI. All static checks and provider tests are now passing. -- 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]
