aa3pankaj edited a comment on pull request #21294: URL: https://github.com/apache/airflow/pull/21294#issuecomment-1029113190
looks like we have few changes in create_db, update_user_auth_stat compared to flask-appbuilder's base class, this could be the reason to move FAB's base class to airflow. But, why don't we have AirflowBaseSecurityManager instead of moving FAB's base class to airflow? something like: from flask_appbuilder.security.manager import BaseSecurityManager class AirflowBaseSecurityManager(BaseSecurityManager) -- 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]
