[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1334485212 ## airflow/www/views.py: ## @@ -1313,23 +1290,12 @@ def last_dagruns(self, session: Session = NEW_SESSION): return flask.json.jsonify(resp)

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1323120010 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -87,35 +132,216 @@ def is_logged_in(self) -> bool: """Return whether the user is logged in."""

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1319941307 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -87,35 +132,216 @@ def is_logged_in(self) -> bool: """Return whether the user is logged in."""

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1318645914 ## airflow/auth/managers/models/resource_method.py: ## @@ -0,0 +1,38 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1318619327 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -87,35 +131,231 @@ def is_logged_in(self) -> bool: """Return whether the user is logged in."""

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1317858875 ## airflow/auth/managers/base_auth_manager.py: ## @@ -102,3 +154,8 @@ def security_manager(self, security_manager: AirflowSecurityManager): :param

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1317858529 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1317392293 ## airflow/auth/managers/base_auth_manager.py: ## @@ -102,3 +154,8 @@ def security_manager(self, security_manager: AirflowSecurityManager): :param

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1309104106 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1307665731 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1307665731 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1306146100 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1306055807 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1306045944 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1306045944 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1306020079 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1305941953 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1305851514 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1305746442 ## airflow/auth/managers/base_auth_manager.py: ## @@ -63,6 +68,69 @@ def get_user_id(self) -> str: def is_logged_in(self) -> bool: """Return whether

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r130428 ## airflow/auth/managers/models/resource_details.py: ## @@ -0,0 +1,31 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1304884883 ## airflow/auth/managers/base_auth_manager.py: ## @@ -63,6 +68,69 @@ def get_user_id(self) -> str: def is_logged_in(self) -> bool: """Return whether

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1304884086 ## airflow/auth/managers/models/resource_action.py: ## @@ -0,0 +1,38 @@ +# +# Licensed to the Apache Software Foundation (ASF) under one +# or more contributor

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1304883915 ## airflow/auth/managers/base_auth_manager.py: ## @@ -63,6 +68,69 @@ def get_user_id(self) -> str: def is_logged_in(self) -> bool: """Return whether

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1295119967 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -17,13 +17,37 @@ # under the License. from __future__ import annotations +import itertools + from flask

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1295078542 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -17,13 +17,37 @@ # under the License. from __future__ import annotations +import itertools + from flask

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1295115248 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -17,13 +17,37 @@ # under the License. from __future__ import annotations +import itertools + from flask

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1295081179 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -81,3 +148,35 @@ def get_url_user_profile(self) -> str | None: if not

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1295078542 ## airflow/auth/managers/fab/fab_auth_manager.py: ## @@ -17,13 +17,37 @@ # under the License. from __future__ import annotations +import itertools + from flask

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1294952463 ## airflow/auth/managers/base_auth_manager.py: ## @@ -54,6 +58,67 @@ def get_user_id(self) -> str: def is_logged_in(self) -> bool: """Return whether

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1294951943 ## airflow/auth/managers/base_auth_manager.py: ## @@ -54,6 +58,67 @@ def get_user_id(self) -> str: def is_logged_in(self) -> bool: """Return whether

[GitHub] [airflow] vincbeck commented on a diff in pull request #33213: Implement `is_authorized()` in auth manager

vincbeck commented on code in PR #33213: URL: https://github.com/apache/airflow/pull/33213#discussion_r1294751595 ## airflow/auth/managers/base_auth_manager.py: ## @@ -54,6 +58,67 @@ def get_user_id(self) -> str: def is_logged_in(self) -> bool: """Return whether