You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/08/13 22:15:13 UTC

[GitHub] [airflow] jhtimmins commented on pull request #10308: Adds a role based authentication backend for the experimental API

jhtimmins commented on pull request #10308:
URL: https://github.com/apache/airflow/pull/10308#issuecomment-673736854


   > What do you think to add role support for all auth backend? I don't understand why you want to do it on the authentication level and not as a separate decorator that will be responsible for permissions handling.
   
   We did it this way for the sake of simplicity. Rather than adding an additional hook for an API that's going to be deprecated shortly, this lets us hook into existing functionality in a non-disruptive way.


----------------------------------------------------------------
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org