rjtshrm commented on issue #33586: URL: https://github.com/apache/airflow/issues/33586#issuecomment-1690033003
After lot of time spent and debugging found that in earlier version we were using **FAB_SECURITY_MANAGER_CLASS** to override the custom auth. But here it is expecting it to SECURITY_MANAGER_CLASS. https://github.com/apache/airflow/blob/2ae1c10bfa59d1b2b5d8d453cd64d9858cb18c0b/airflow/www/extensions/init_appbuilder.py#L657 As it can be seen that if SECURITY_MANAGER_CLASS is not set then it uses default AirflowSecurityManager where the custom auth login for aws cognito was missing and hence was getting the error. Although in the airflow 2.7.0 docs https://airflow.apache.org/docs/apache-airflow/stable/security/webserver.html, FAB_SECURITY_MANAGER_CLASS is mentioned. Also, make sure SECURITY_MANAGER_CLASS to class type not a string as in FAB_SECURITY_MANAGER_CLASS. @potiuk @vincbeck Can you confirm this behaviour or am i missing something -- 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]
