[PR] Implement `is_authorized_variable` in AWS auth manager [airflow]

2023-11-22 Thread via GitHub
vincbeck opened a new pull request, #35804: URL: https://github.com/apache/airflow/pull/35804 Leverage Amazon Verified Permissions to check user permissions related to variables in AWS auth manager. This PR is the first one to implement a `is_authorize_*` method in AWS auth manager, and as

Re: [PR] Implement `is_authorized_variable` in AWS auth manager [airflow]

2023-11-23 Thread via GitHub
o-nikolas commented on code in PR #35804: URL: https://github.com/apache/airflow/pull/35804#discussion_r1403683906 ## airflow/providers/amazon/aws/auth_manager/avp/facade.py: ## @@ -0,0 +1,125 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributo

Re: [PR] Implement `is_authorized_variable` in AWS auth manager [airflow]

2023-11-23 Thread via GitHub
vincbeck commented on code in PR #35804: URL: https://github.com/apache/airflow/pull/35804#discussion_r1403761025 ## airflow/providers/amazon/aws/auth_manager/avp/facade.py: ## @@ -0,0 +1,125 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

Re: [PR] Implement `is_authorized_variable` in AWS auth manager [airflow]

2023-11-27 Thread via GitHub
o-nikolas commented on code in PR #35804: URL: https://github.com/apache/airflow/pull/35804#discussion_r1406692649 ## airflow/providers/amazon/aws/auth_manager/avp/entities.py: ## @@ -0,0 +1,57 @@ +# Licensed to the Apache Software Foundation (ASF) under one +# or more contribut

Re: [PR] Implement `is_authorized_variable` in AWS auth manager [airflow]

2023-11-27 Thread via GitHub
vincbeck merged PR #35804: URL: https://github.com/apache/airflow/pull/35804 -- 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-unsubscr...@airflow