Skip to content

Commit dba0ebc

Browse files
Make BaseAuthManager an abstract base class (apache#47692)
We have a number of abstract methods, but we weren't using the ABCMeta metaclass, so we weren't actually checking auth managers implement them.
1 parent 45d35e2 commit dba0ebc

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

airflow/api_fastapi/auth/managers/base_auth_manager.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
from __future__ import annotations
1919

2020
import logging
21-
from abc import abstractmethod
21+
from abc import ABCMeta, abstractmethod
2222
from typing import TYPE_CHECKING, Any, Generic, TypeVar
2323

2424
from jwt import InvalidTokenError
@@ -66,7 +66,7 @@
6666
T = TypeVar("T", bound=BaseUser)
6767

6868

69-
class BaseAuthManager(Generic[T], LoggingMixin):
69+
class BaseAuthManager(Generic[T], LoggingMixin, metaclass=ABCMeta):
7070
"""
7171
Class to derive in order to implement concrete auth managers.
7272

0 commit comments

Comments
 (0)