From 3fe3bfad9705de0d98b0d17d76828be2cd0ee71b Mon Sep 17 00:00:00 2001 From: Mohammad Torkashvand Date: Fri, 7 Jun 2024 09:43:23 +0200 Subject: [PATCH] make user required again --- oauth2_lib/fastapi.py | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/oauth2_lib/fastapi.py b/oauth2_lib/fastapi.py index 27c7c43..bcf3262 100644 --- a/oauth2_lib/fastapi.py +++ b/oauth2_lib/fastapi.py @@ -263,7 +263,7 @@ class Authorization(ABC): """ @abstractmethod - async def authorize(self, request: HTTPConnection, user: Optional[OIDCUserModel] = None) -> Optional[bool]: + async def authorize(self, request: HTTPConnection, user: OIDCUserModel) -> Optional[bool]: pass @@ -274,7 +274,7 @@ class GraphqlAuthorization(ABC): """ @abstractmethod - async def authorize(self, request: RequestPath, user: Optional[OIDCUserModel] = None) -> Optional[bool]: + async def authorize(self, request: RequestPath, user: OIDCUserModel) -> Optional[bool]: pass @@ -324,7 +324,7 @@ class OPAAuthorization(Authorization, OPAMixin): Uses OAUTH2 settings and request information to authorize actions. """ - async def authorize(self, request: HTTPConnection, user_info: Optional[OIDCUserModel] = None) -> Optional[bool]: + async def authorize(self, request: HTTPConnection, user_info: OIDCUserModel) -> Optional[bool]: if not (oauth2lib_settings.OAUTH2_ACTIVE and oauth2lib_settings.OAUTH2_AUTHORIZATION_ACTIVE): return None @@ -380,7 +380,7 @@ def __init__(self, opa_url: str, auto_error: bool = False, opa_kwargs: Union[Map # By default don't raise HTTP 403 because partial results are preferred super().__init__(opa_url, auto_error, opa_kwargs) - async def authorize(self, request: RequestPath, user_info: Optional[OIDCUserModel] = None) -> Optional[bool]: + async def authorize(self, request: RequestPath, user_info: OIDCUserModel) -> Optional[bool]: if not (oauth2lib_settings.OAUTH2_ACTIVE and oauth2lib_settings.OAUTH2_AUTHORIZATION_ACTIVE): return None