vincbeck commented on code in PR #69790:
URL: https://github.com/apache/airflow/pull/69790#discussion_r3614897655
##########
airflow-core/src/airflow/api_fastapi/auth/managers/base_auth_manager.py:
##########
@@ -365,6 +365,28 @@ def is_authorized_view(
:param user: the user to performing the action
"""
+ def is_authorized_view_for_team(
Review Comment:
> We could instead fold it into is_authorized_view with an optional
team_name and take it as a breaking change for out-of-band AuthManagers, but
I'd prefer to avoid that. WDYT?
To be honest I think I rather having the breaking change. The method
`is_authorized_view_for_team` feels wrong to me. If we had done the same for
the other methods then we would have duplicated all the methods in the auth
manager interface
--
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]
