From 2a25cce03405d23a19f984586aab3dd341912fd5 Mon Sep 17 00:00:00 2001 From: Frederick Muriuki Muriithi Date: Mon, 13 May 2024 06:33:51 +0300 Subject: Fix myriad of linting error These linting errors can't be rebased into the newer commits. --- gn_auth/auth/authentication/oauth2/grants/jwt_bearer_grant.py | 6 ++++-- gn_auth/auth/authentication/oauth2/models/oauth2client.py | 4 ++-- gn_auth/auth/authentication/oauth2/server.py | 4 ++-- 3 files changed, 8 insertions(+), 6 deletions(-) (limited to 'gn_auth/auth/authentication/oauth2') diff --git a/gn_auth/auth/authentication/oauth2/grants/jwt_bearer_grant.py b/gn_auth/auth/authentication/oauth2/grants/jwt_bearer_grant.py index d663aaf..b0f2cc7 100644 --- a/gn_auth/auth/authentication/oauth2/grants/jwt_bearer_grant.py +++ b/gn_auth/auth/authentication/oauth2/grants/jwt_bearer_grant.py @@ -35,8 +35,10 @@ class JWTBearerTokenGenerator(_JWTBearerTokenGenerator): } - def __call__(self, grant_type, client, user=None, scope=None, - expires_in=None, include_refresh_token=True): + def __call__(# pylint: disable=[too-many-arguments] + self, grant_type, client, user=None, scope=None, expires_in=None, + include_refresh_token=True + ): # there is absolutely no refresh token in JWT format """ The default generator does not provide refresh tokens with JWT. It goes diff --git a/gn_auth/auth/authentication/oauth2/models/oauth2client.py b/gn_auth/auth/authentication/oauth2/models/oauth2client.py index f48cbce..d31faf6 100644 --- a/gn_auth/auth/authentication/oauth2/models/oauth2client.py +++ b/gn_auth/auth/authentication/oauth2/models/oauth2client.py @@ -60,8 +60,8 @@ class OAuth2Client(ClientMixin): @cached_property def jwks(self) -> KeySet: """Return this client's KeySet.""" - def __parse_key__(keypath: Path) -> JsonWebKey:# pylint: disable=[unspecified-encoding] - with open(keypath) as _key: + def __parse_key__(keypath: Path) -> JsonWebKey: + with open(keypath) as _key:# pylint: disable=[unspecified-encoding] return JsonWebKey.import_key(_key.read()) return KeySet([ diff --git a/gn_auth/auth/authentication/oauth2/server.py b/gn_auth/auth/authentication/oauth2/server.py index 75d6e1b..d845c60 100644 --- a/gn_auth/auth/authentication/oauth2/server.py +++ b/gn_auth/auth/authentication/oauth2/server.py @@ -11,7 +11,7 @@ from authlib.integrations.flask_oauth2 import AuthorizationServer from gn_auth.auth.db import sqlite3 as db -from .models.oauth2client import client +from .models.oauth2client import client as fetch_client from .models.oauth2token import OAuth2Token, save_token from .models.jwtrefreshtoken import ( JWTRefreshToken, @@ -36,7 +36,7 @@ def create_query_client_func() -> Callable: # use current_app rather than passing the db_uri to avoid issues # when config changes, e.g. while testing. with db.connection(current_app.config["AUTH_DB"]) as conn: - _client = client(conn, client_id).maybe( + _client = fetch_client(conn, client_id).maybe( None, lambda clt: clt) # type: ignore[misc] if bool(_client): return _client -- cgit v1.2.3