Add name to handlers and refactor.

Author: everaldorodrigo

src/handlers/api.py

@@ -48,21 +48,14 @@ class BaseHandler(BioThingsAuthnMixin, BaseAPIHandler):
 
 class AuthHandler(BaseHandler):
     def set_cache_header(self, cache_value):
-        # disabel cache for auth-related handlers
+        # disable cache for auth-related handlers
         self.set_header("Cache-Control", "private, max-age=0, no-cache")
 
-    def prepare(self):
-        """Override prepare to bypass parameter validation issues"""
-        # codacy:disable
-        super(BaseAPIHandler, self).prepare()
-
 
 class UserInfoHandler(AuthHandler):
-    """ "Handler for /user_info endpoint."""
-
-    # Define that no parameters are required for this endpoint
+    """ "Handler for /user endpoint."""
+    name = "user_info"
     kwargs = {
-        "*": {},  # Override any inherited parameter requirements
         "GET": {}  # Explicitly empty - no parameters expected or required
     }
 
@@ -84,9 +77,9 @@ def get(self):
 
 
 class LoginHandler(AuthHandler):
-    # Define expected parameters for login redirect
+    """ "Handler for /login endpoint."""
+    name = "user_login"
     kwargs = {
-        "*": {},  # Override any inherited parameter requirements
         "GET": {
             "next": {"type": str, "required": False, "default": "/"}  # Redirect URL
         }
@@ -97,9 +90,9 @@ def get(self):
 
 
 class LogoutHandler(AuthHandler):
-    # Define expected parameters for logout redirect
+    """ "Handler for /logout endpoint."""
+    name = "user_logout"
     kwargs = {
-        "*": {},  # Override any inherited parameter requirements
         "GET": {
             "next": {"type": str, "required": False, "default": "/"}  # Redirect URL
         }

src/handlers/oauth.py

@@ -7,26 +7,12 @@
 
 
 class GitHubLoginHandler(BaseAPIHandler, GithubOAuth2Mixin):
-    """ "Handler for GitHub oauth login"""
+    """ "Handler for GitHub oauth login: /oauth endpoint"""
+    name = "github_oauth"
 
     SCOPES = []
     GITHUB_CALLBACK_PATH = "/oauth"
 
-    # Define expected parameters properly - override any inherited parameter validation
-    kwargs = {
-        "*": {},  # Override any inherited parameter requirements
-        "GET": {
-            "code": {"type": str, "required": False},  # OAuth callback code
-            "next": {"type": str, "required": False, "default": "/"},  # Redirect URL
-            "state": {"type": str, "required": False},  # OAuth state parameter
-        }
-    }
-
-    def prepare(self):
-        """Override prepare to bypass parameter validation issues"""
-        # codacy:disable
-        super(BaseAPIHandler, self).prepare()
-
     async def get(self):
         CLIENT_ID = self.biothings.config.GITHUB_CLIENT_ID
         CLIENT_SECRET = self.biothings.config.GITHUB_CLIENT_SECRET