feng-tao edited a comment on issue #4612: [AIRFLOW-3785] Decommission 
airflow/contrib/auth/backends
URL: https://github.com/apache/airflow/pull/4612#issuecomment-459041879
 
 
   @XD-DENG , I would suggest we do it in multiple small PRs instead of one 
single giant PR which is to help us to revert in case anything happens.
   
   In terms of this pr, I think FAB supports LDAP / OATH 
authentication(https://flask-appbuilder.readthedocs.io/en/latest/security.html#supported-authentication-types)
 which has been used by lyft's superset service in production. Hence I think we 
could remove those. For password_auth, I think it is worth to check in the 
email alias and see if anyone still depends on it. If it turns out that many 
people are still using it, we should refactor for FAB's  ab_user table. What do 
you think?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
[email protected]


With regards,
Apache Git Services

Reply via email to