This is an automated email from the ASF dual-hosted git repository.
ephraimanierobi pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 8387294d66 Remove `require_authentication` decorator from security.py
(#35340)
8387294d66 is described below
commit 8387294d66fa8d84a27da3b62ff7d4989d707cad
Author: Ephraim Anierobi <splendidzig...@gmail.com>
AuthorDate: Wed Nov 1 16:42:29 2023 +0100
Remove `require_authentication` decorator from security.py (#35340)
This decorator was added in #34317, but it seems it's not used anywhere.
Opening this PR to see if I'm missing something
---
airflow/api_connexion/security.py | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/airflow/api_connexion/security.py
b/airflow/api_connexion/security.py
index 6da171aa62..2896ec2362 100644
--- a/airflow/api_connexion/security.py
+++ b/airflow/api_connexion/security.py
@@ -92,17 +92,6 @@ def _requires_access(*, is_authorized_callback: Callable[[],
bool], func: Callab
raise PermissionDenied()
-def requires_authentication(func: T):
- """Decorator for functions that require authentication."""
-
- @wraps(func)
- def decorated(*args, **kwargs):
- check_authentication()
- return func(*args, **kwargs)
-
- return cast(T, decorated)
-
-
def requires_access_configuration(method: ResourceMethod) -> Callable[[T], T]:
def requires_access_decorator(func: T):
@wraps(func)
