Re: [PR] Implement `batch_is_authorized_*` APIs in AWS auth manager [airflow]

2024-02-22 Thread via GitHub
vincbeck merged PR #37430: URL: https://github.com/apache/airflow/pull/37430 -- 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:

Re: [PR] Implement `batch_is_authorized_*` APIs in AWS auth manager [airflow]

2024-02-21 Thread via GitHub
o-nikolas commented on code in PR #37430: URL: https://github.com/apache/airflow/pull/37430#discussion_r1498351268 ## airflow/providers/amazon/aws/auth_manager/avp/facade.py: ## @@ -116,6 +125,63 @@ def is_authorized( return resp["decision"] == "ALLOW" +def

Re: [PR] Implement `batch_is_authorized_*` APIs in AWS auth manager [airflow]

2024-02-21 Thread via GitHub
vincbeck commented on code in PR #37430: URL: https://github.com/apache/airflow/pull/37430#discussion_r1498347166 ## airflow/providers/amazon/aws/auth_manager/avp/facade.py: ## @@ -116,6 +125,63 @@ def is_authorized( return resp["decision"] == "ALLOW" +def

Re: [PR] Implement `batch_is_authorized_*` APIs in AWS auth manager [airflow]

2024-02-21 Thread via GitHub
o-nikolas commented on code in PR #37430: URL: https://github.com/apache/airflow/pull/37430#discussion_r1498340343 ## airflow/auth/managers/base_auth_manager.py: ## @@ -254,6 +254,24 @@ def is_authorized_custom_view( """ raise AirflowException(f"The resource

[PR] Implement `batch_is_authorized_*` APIs in AWS auth manager [airflow]

2024-02-14 Thread via GitHub
vincbeck opened a new pull request, #37430: URL: https://github.com/apache/airflow/pull/37430 Implement `batch_is_authorized_*` APIs in AWS auth manager --- **^ Add meaningful description above** Read the **[Pull Request