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 2021/09/14 14:23:17 UTC

[GitHub] [airflow] jedcunningham edited a comment on pull request #18222: make current_user_has_permissions backwards compatible

jedcunningham edited a comment on pull request #18222:
URL: https://github.com/apache/airflow/pull/18222#issuecomment-919193243


   I mean, both Airflow and FAB overall assume permissions come from roles. If someone is trying to subclass away roles, seems reasonable to me to need to adjust a few new methods every so often 🤷‍♂️. It was written as it is so it can short-circuit vs merging all permissions if a user has multiple roles.
   
   @jhtimmins, what are your thoughts here?


-- 
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: commits-unsubscribe@airflow.apache.org

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