fix: removed redundant caching in server extension

Author: florian-jaeger

grader_labextension/handlers/permission.py

@@ -6,7 +6,7 @@
 
 import json
 from grader_labextension.registry import register_handler
-from grader_labextension.handlers.base_handler import ExtensionBaseHandler, cache
+from grader_labextension.handlers.base_handler import ExtensionBaseHandler
 from tornado.web import HTTPError, authenticated
 from grader_labextension.services.request import RequestServiceError
 
@@ -17,8 +17,6 @@ class PermissionBaseHandler(ExtensionBaseHandler):
     Tornado Handler class for http requests to /permissions.
     """
     @authenticated
-    @cache(max_age=10)
-    @authenticated
     async def get(self):
         """ Sends a GET-request to the grader service and returns the permissions of a user
         """

grader_labextension/services/request.py

@@ -6,7 +6,6 @@
 from tornado.httpclient import AsyncHTTPClient, HTTPRequest, HTTPResponse, HTTPError
 from traitlets import Unicode, TraitError, validate
 from traitlets.config import SingletonConfigurable
-from cachetools import TTLCache
 
 class RequestServiceError(Exception):
     def __init__(self, code: int, status_text: str, message: str):
@@ -26,8 +25,6 @@ def __init__(
         self, 
         default_request_timeout: float = 20.0, 
         default_connect_timeout: float = 20.0, 
-        cache_ttl: int = 60,  # Time-to-live for cached responses in seconds
-        max_cache_size: int = 100,  # Maximum number of cached responses
         max_retries: int = 3,  # Max retry attempts for transient errors
         **kwargs
     ):
@@ -37,7 +34,6 @@ def __init__(
         self.default_request_timeout = default_request_timeout
         self.default_connect_timeout = default_connect_timeout
         self.max_retries = max_retries
-        self.cache = TTLCache(maxsize=max_cache_size, ttl=cache_ttl)
 
     def get_authorization_header(self):
         auth_token = os.environ.get("GRADER_API_TOKEN")
@@ -115,11 +111,6 @@ async def request(
         if isinstance(body, dict):
             body = json.dumps(body)
 
-        cache_key = f"{method}:{endpoint}"
-        if method == "GET" and cache_key in self.cache:
-            self.log.info(f"Returning cached response for {endpoint}")
-            return self.cache[cache_key]
-
         request = HTTPRequest(
             url=self.url + endpoint,
             method=method,
@@ -137,9 +128,6 @@ async def request(
             else:
                 response_data = response
 
-            if method == "GET":
-                self.cache[cache_key] = response_data
-
             if response_callback:
                 response_callback(response)