chore: remove CHANGELOG.md and test files; update skip_limiter decorator for type checking

Author: long2ice

CHANGELOG.md

@@ -17,5 +17,5 @@ async def health():
     async def wrapper(*args, **kwargs):
         return await func(*args, **kwargs)
 
-    wrapper._skip_limiter = True
+    wrapper._skip_limiter = True  # type: ignore
     return wrapper

fastapi_limiter/decorators.py

@@ -29,7 +29,6 @@ def __init__(
 
     async def _check(self, key: str) -> int:
         redis = FastAPILimiter.redis
-        assert redis is not None
         pexpire: int = await redis.evalsha(
             FastAPILimiter.lua_sha, 1, key, str(self.times), str(self.milliseconds)
         )
@@ -45,10 +44,16 @@ async def __call__(self, request: Request, response: Response):
         route_index = 0
         dep_index = 0
         for i, route in enumerate(request.app.routes):
-            if route.path == request.scope["path"] and hasattr(route, "methods") and request.method in route.methods:
+            if (
+                route.path == request.scope["path"]
+                and hasattr(route, "methods")
+                and request.method in route.methods
+            ):
                 route_index = i
                 # Check if the route endpoint has _skip_limiter attribute
-                if hasattr(route, "endpoint") and getattr(route.endpoint, "_skip_limiter", False):
+                if hasattr(route, "endpoint") and getattr(
+                    route.endpoint, "_skip_limiter", False
+                ):
                     return
                 for j, dependency in enumerate(route.dependencies):
                     if self is dependency.dependency:
@@ -58,8 +63,6 @@ async def __call__(self, request: Request, response: Response):
         # moved here because constructor run before app startup
         identifier = self.identifier or FastAPILimiter.identifier
         callback = self.callback or FastAPILimiter.http_callback
-        assert identifier is not None
-        assert callback is not None
         rate_key = await identifier(request)
         key = f"{FastAPILimiter.prefix}:{rate_key}:{route_index}:{dep_index}"
         try:
@@ -81,7 +84,6 @@ async def __call__(self, ws: WebSocket, context_key: str = ""):
                 "You must call FastAPILimiter.init in startup event of fastapi!"
             )
         identifier = self.identifier or FastAPILimiter.identifier
-        assert identifier is not None
         rate_key = await identifier(ws)
         key = f"{FastAPILimiter.prefix}:ws:{rate_key}:{context_key}"
         pexpire = await self._check(key)

fastapi_limiter/depends.py

@@ -65,3 +65,11 @@ def test_limiter_websockets():
             data = ws.receive_text()
             assert data == "Hello, world"
             ws.close()
+
+
+def test_skip_limiter():
+    with TestClient(app) as client:
+        # Even with RateLimiter(times=1), skip_limiter allows unlimited requests
+        for _ in range(5):
+            response = client.get("/skip")
+            assert response.status_code == 200