mik-laj commented on a change in pull request #10594:
URL: https://github.com/apache/airflow/pull/10594#discussion_r482790126
##########
File path: airflow/api_connexion/security.py
##########
@@ -37,3 +37,32 @@ def decorated(*args, **kwargs):
return function(*args, **kwargs)
return cast(T, decorated)
+
+
+def requires_access(permissions: Sequence[Tuple[str, str]]) -> Callable[[T],
T]:
Review comment:
Config endpoint should have authentication and authorization.
Health and version endpoint don't need any protection. There are no
sensitive data there and for operational reasons it is better not to add them.
This will greatly facilitate monitoring, autodiscovery and API client
development.
##########
File path: airflow/api_connexion/security.py
##########
@@ -37,3 +37,32 @@ def decorated(*args, **kwargs):
return function(*args, **kwargs)
return cast(T, decorated)
+
+
+def requires_access(permissions: Sequence[Tuple[str, str]]) -> Callable[[T],
T]:
Review comment:
Config endpoint should have authentication and authorization.
Health and version endpoint don't need any protection. There are no
sensitive data there and for operational reasons it is better not to add them.
This will greatly facilitate monitoring, service autodiscovery and API client
development.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
