My mistake, yes it's just for UI. Not for API. I'll be quiet now :-)
On Thu, Jan 31, 2019 at 1:42 PM Deng Xiaodong <xd.den...@gmail.com> wrote: > Hi Shah, > > Thanks for your reply. > > May I confirm that you mean that you’re using it for the UI or for the API > authentication? What we discuss here is only the API authentication. > > If you meant UI authentication, actually the non-RBAC UI is already > deprecated in master branch, as I shared (meaning from one of the future > release, only FAB-based RBAC UI will be supported). > > > XD > > > On 31 Jan 2019, at 9:37 PM, Shah Altaf <mend...@gmail.com> wrote: > > > > Hello, yes we are using password_auth for all of our Airflow > > installations. We specifically use it to automate creation of many > users. > > > > > > > > On Thu, Jan 31, 2019 at 1:08 PM Deng Xiaodong <xd.den...@gmail.com> > wrote: > > > >> Hi folks, > >> > >> As you may have noticed, the Flask-Admin based UI (non-RBAC) was already > >> deprecated in the master branch. Some works are going on to further > clean > >> the codebase. > >> > >> In the process, we found that there are some “legacy" modules in > >> “airflow/contrib/auth/backends/ < > >> > https://github.com/apache/airflow/pull/4612/files#diff-fa19b36f43ec33307cd3ffe32f8b5397 > >“. > >> Most of them should be removed directly, as they’re no longer applicable > >> for the new UI. But there is one module, > >> “airflow/contrib/auth/backends/password_auth.py < > >> > https://github.com/apache/airflow/pull/4612/files#diff-fa19b36f43ec33307cd3ffe32f8b5397 > >”, > >> is used for API authentication as well. > >> > >> We would like to understand from the community, if anyone is using > >> airflow/contrib/auth/backends/password_auth.py < > >> > https://github.com/apache/airflow/pull/4612/files#diff-fa19b36f43ec33307cd3ffe32f8b5397 > > > >> for API authentication? This will help estimate & decide whether we > remove > >> it directly or spend some effort to refactor it. > >> > >> Thanks for your inputs in advance. > >> > >> > >> XD > >> > >> Reference: https://github.com/apache/airflow/pull/4612 < > >> https://github.com/apache/airflow/pull/4612> > >
