Re: [PR] Implement `is_authorized()` in auth manager [airflow]

["utkarsharma2 (via GitHub)" <gi...@apache.org>]

utkarsharma2 commented on code in PR #33213:
URL: https://github.com/apache/airflow/pull/33213#discussion_r1343788762


##########
airflow/www/auth.py:
##########
@@ -35,16 +44,100 @@ def get_access_denied_message():
     return conf.get("webserver", "access_denied_message")
 
 
-def has_access(permissions: Sequence[tuple[str, str]] | None = None) -> Callable[[T], T]:

Review Comment:
   @vincbeck This is a breaking change for us, we have a plugin that uses this method. Can we please keep this method and work around it?



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