feat: closes #452: delete method support for memcached backend (manual merge)

fix #430: Added typehint for Callable[P, R] in the decorator

.github/workflows/ci-cd.yml

@@ -3,7 +3,6 @@ on:
   push:
     branches:
       - main
-      - 'release-*'
     tags:
       - 'v*'
   pull_request:
@@ -15,7 +14,7 @@ jobs:
     name: Linter
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Install Poetry
         run: pipx install poetry
       - name: Setup Python
@@ -25,7 +24,7 @@ jobs:
           python-version: '3.x'
           cache: poetry
       - name: Cache mypy cache
-        uses: actions/cache@v4
+        uses: actions/cache@v3
         with:
           path: .mypy_cache
           key: ${{ runner.os }}-mypy-${{ steps.setup-python.outputs.python-version }}-${{ hashFiles('poetry.lock') }}
@@ -49,7 +48,7 @@ jobs:
     runs-on: ubuntu-latest
 
     steps:
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Install Poetry
         run: pipx install poetry
       - uses: actions/setup-python@v5
@@ -76,12 +75,11 @@ jobs:
 
   build:
     name: Build distributions
-    if: github.event_name == 'push' && github.ref == 'refs/heads/main'
     runs-on: ubuntu-latest
     needs: [test-summary]
     steps:
       - name: Checkout
-        uses: actions/checkout@v4
+        uses: actions/checkout@v3
       - name: Install Poetry
         run: pipx install poetry
       - name: Setup Python
@@ -93,7 +91,7 @@ jobs:
         run:
           make build
       - name: Upload artifacts
-        uses: actions/upload-artifact@v4
+        uses: actions/upload-artifact@v3
         with:
           name: dist
           path: dist
@@ -112,7 +110,7 @@ jobs:
       id-token: write
     steps:
       - name: Download artifacts
-        uses: actions/download-artifact@v4
+        uses: actions/download-artifact@v2
         with:
           name: dist
           path: dist

.github/workflows/codeql.yml

@@ -20,15 +20,15 @@ jobs:
 
     steps:
     - name: Checkout repository
-      uses: actions/checkout@v4
+      uses: actions/checkout@v3
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@v3
+      uses: github/codeql-action/init@v2
       with:
         languages: python
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@v3
+      uses: github/codeql-action/analyze@v2
       with:
         category: "/language:python"

.github/workflows/dependabot-auto-merge.yml

@@ -12,9 +12,9 @@ jobs:
     steps:
       - name: Dependabot metadata
         id: dependabot-metadata
-        uses: dependabot/fetch-metadata@v2
+        uses: dependabot/fetch-metadata@v1
 
-      - uses: actions/checkout@v4
+      - uses: actions/checkout@v3
       - name: Approve PR
         # only auto-approve direct deps that are minor or patch updates
         #   dependency type is indirect, direct:development or direct:production

.github/workflows/towncrier.yml

@@ -23,7 +23,7 @@ jobs:
         || contains(github.event.pull_request.labels.*.name, 'skip-changelog')
       )
     steps:
-    - uses: actions/checkout@v4
+    - uses: actions/checkout@v2
     - name: Install Poetry
       run: pipx install poetry
     - name: Setup Python

CHANGELOG.md

@@ -8,27 +8,6 @@ This project uses [*towncrier*](https://towncrier.readthedocs.io/) and the chang
 
 <!-- towncrier release notes start -->
 
-## [0.2.2](https://github.com/long2ice/fastapi-cache/tree/0.2.2) - 2025-01-18
-
-### Bug Fixes
-
-- Fix failing tests (#459) [#459](https://github.com/long2ice/fastapi-cache/issues/459)
-
-### Build Changes
-
-- Use `importlib.metadata` to include project version string as `fastapi_cache.__version__`. [#172](https://github.com/long2ice/fastapi-cache/issues/172)
-- (dependabot) Bump actions/checkout from 2 to 4 [#293](https://github.com/long2ice/fastapi-cache/issues/293)
-- (dependabot) Bump actions/download-artifact from 2 to 4 (#359) [#359](https://github.com/long2ice/fastapi-cache/issues/359)
-- (dependabot) Bump actions/upload-artifact from 3 to 4 (#360) [#360](https://github.com/long2ice/fastapi-cache/issues/360)
-- (dependabot) Bump actions/cache from 3 to 4 (#378) [#378](https://github.com/long2ice/fastapi-cache/issues/378)
-- (dependabot) Bump dependabot/fetch-metadata from 1 to 2 (#464) [#464](https://github.com/long2ice/fastapi-cache/issues/464)
-- (dependabot) Bump tox from 4.20.0 to 4.23.2 (#466) [#466](https://github.com/long2ice/fastapi-cache/issues/466)
-- (dependabot) Bump fastapi from 0.115.0 to 0.115.6 (#486) [#486](https://github.com/long2ice/fastapi-cache/issues/486)
-- (dependabot) Bump redis from 5.0.8 to 5.2.1 (#490) [#490](https://github.com/long2ice/fastapi-cache/issues/490)
-- (dependabot) Bump uvicorn from 0.30.6 to 0.33.0 (#493) [#493](https://github.com/long2ice/fastapi-cache/issues/493)
-- (dependabot) Bump pyright from 1.1.381 to 1.1.392.post0 (#507) [#507](https://github.com/long2ice/fastapi-cache/issues/507)
-- (dependabot) Bump towncrier from 22.12.0 to 24.8.0 (#509) [#509](https://github.com/long2ice/fastapi-cache/issues/509)
-
 ## 0.2
 
 ### 0.2.1

README.md

@@ -50,13 +50,42 @@ or
 
 ### Quick Start
 
-```python
-from collections.abc import AsyncIterator
-from contextlib import asynccontextmanager
+Using in-memory backend:
 
+```python
+from contextlib import asynccontextmanager
+from fastapi import FastAPI
+
+from fastapi_cache import FastAPICache
+from fastapi_cache.backends.inmemory import InMemoryBackend
+from fastapi_cache.decorator import cache
+
+
+@asynccontextmanager
+async def lifespan(app: FastAPI):
+    FastAPICache.init(InMemoryBackend(), prefix="fastapi-cache")
+    yield
+
+
+app = FastAPI(lifespan=lifespan)
+
+
+@cache()
+async def get_cache():
+    return 1
+
+
+@app.get("/")
+@cache(expire=60)
+async def index():
+    return dict(hello="world")
+```
+
+or [Redis](https://redis.io) backend:
+
+```python
+from contextlib import asynccontextmanager
 from fastapi import FastAPI
-from starlette.requests import Request
-from starlette.responses import Response
 
 from fastapi_cache import FastAPICache
 from fastapi_cache.backends.redis import RedisBackend
@@ -66,7 +95,7 @@ from redis import asyncio as aioredis
 
 
 @asynccontextmanager
-async def lifespan(_: FastAPI) -> AsyncIterator[None]:
+async def lifespan(app: FastAPI):
     redis = aioredis.from_url("redis://localhost")
     FastAPICache.init(RedisBackend(redis), prefix="fastapi-cache")
     yield
@@ -85,7 +114,6 @@ async def get_cache():
 async def index():
     return dict(hello="world")
 ```
-
 ### Initialization
 
 First you must call `FastAPICache.init` during startup FastAPI startup; this is where you set global configuration.

changelog.d/172.feature.md

@@ -0,0 +1 @@
+Use `importlib.metadata` to include project version string as `fastapi_cache.__version__`.

changelog.d/352.feature.md

@@ -0,0 +1 @@
+Delete method support for memcached backend (from @xodiumx)

fastapi_cache/backends/dynamodb.py

@@ -1,5 +1,5 @@
 import datetime
-from typing import TYPE_CHECKING, Optional, Tuple, Union
+from typing import TYPE_CHECKING, Optional, Tuple
 
 from aiobotocore.client import AioBaseClient
 from aiobotocore.session import AioSession, get_session
@@ -30,7 +30,7 @@ class DynamoBackend(Backend):
         >> FastAPICache.init(dynamodb)
     """
 
-    client: Union[DynamoDBClient, None]
+    client: DynamoDBClient
     session: AioSession
     table_name: str
     region: Optional[str]
@@ -46,12 +46,9 @@ class DynamoBackend(Backend):
         ).__aenter__()
 
     async def close(self) -> None:
-        if self.client:
-            await self.client.__aexit__(None, None, None)
-            self.client = None
+        self.client = await self.client.__aexit__(None, None, None)
 
     async def get_with_ttl(self, key: str) -> Tuple[int, Optional[bytes]]:
-        if self.client:
         response = await self.client.get_item(TableName=self.table_name, Key={"key": {"S": key}})
 
         if "Item" in response:
@@ -69,14 +66,12 @@ class DynamoBackend(Backend):
         return 0, None
 
     async def get(self, key: str) -> Optional[bytes]:
-        if self.client:
         response = await self.client.get_item(TableName=self.table_name, Key={"key": {"S": key}})
         if "Item" in response:
             return response["Item"].get("value", {}).get("B")
         return None
 
     async def set(self, key: str, value: bytes, expire: Optional[int] = None) -> None:
-        if self.client:
         ttl = (
             {
                 "ttl": {

fastapi_cache/backends/memcached.py

@@ -19,4 +19,11 @@ class MemcachedBackend(Backend):
         await self.mcache.set(key.encode(), value, exptime=expire or 0)
 
     async def clear(self, namespace: Optional[str] = None, key: Optional[str] = None) -> int:
-        raise NotImplementedError
+        is_deleted = False
+        if key:
+            is_deleted = await self.mcache.delete(key=key.encode())
+        else:
+            await self.mcache.flush_all()
+            is_deleted = True
+
+        return int(is_deleted)

fastapi_cache/decorator.py

@@ -90,7 +90,7 @@ def cache(
     key_builder: Optional[KeyBuilder] = None,
     namespace: str = "",
     injected_dependency_namespace: str = "__fastapi_cache",
-) -> Callable[[Callable[P, Awaitable[R]]], Callable[P, Awaitable[Union[R, Response]]]]:
+) -> Callable[[Union[Callable[P, Awaitable[R]], Callable[P, R]]], Callable[P, Awaitable[Union[R, Response]]]]:
     """
     cache all function
     :param injected_dependency_namespace:
@@ -114,7 +114,7 @@ def cache(
     )
 
     def wrapper(
-        func: Callable[P, Awaitable[R]]
+        func: Union[Callable[P, Awaitable[R]], Callable[P, R]]
     ) -> Callable[P, Awaitable[Union[R, Response]]]:
         # get_typed_signature ensures that any forward references are resolved first
         wrapped_signature = get_typed_signature(func)
@@ -142,7 +142,7 @@ def cache(
                     # unintuitively, we have to await once here, so that caller
                     # does not have to await twice. See
                     # https://stackoverflow.com/a/59268198/532513
-                    return await func(*args, **kwargs)
+                    return await func(*args, **kwargs)  # type: ignore[no-any-return]
                 else:
                     # sync, wrap in thread and return async
                     # see above why we have to await even although caller also awaits.
@@ -221,7 +221,8 @@ def cache(
                         return response
 
                 result = cast(R, coder.decode_as_type(cached, type_=return_type))
-
+                if response and isinstance(result, Response):
+                    result.headers.update(response.headers)
             return result
 
         inner.__signature__ = _augment_signature(wrapped_signature, *to_inject)  # type: ignore[attr-defined]

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "fastapi-cache2"
-version = "0.2.3"
+version = "0.2.2"
 description = "Cache for FastAPI"
 authors = ["long2ice <long2ice@gmail.com>"]
 license = "Apache-2.0"
@@ -23,17 +23,17 @@ importlib-metadata = { version = "^6.6.0", python = "<3.8" }
 pendulum = "^3.0.0"
 aiomcache = { version = "^0.8.2", optional = true }
 aiobotocore = {version = "^2.13.1", optional = true}
-redis = {version = "^5.0.8", extras = ["redis"]}
+redis = {version = "^5.2.0", extras = ["redis"]}
 
 [tool.poetry.group.linting]
 optional = true
 
 [tool.poetry.group.linting.dependencies]
-mypy = { version = "^1.2.0", python = "^3.10" }
-pyright = { version = "^1.1.373", python = "^3.10" }
+mypy = { version = "^1.13.0", python = "^3.10" }
+pyright = { version = "^1.1.388", python = "^3.10" }
 types-aiobotocore = { extras = ["dynamodb"], version = "^2.5.0.post2", python = "^3.10" }
 types-redis = { version = "^4.5.4.2", python = "^3.10" }
-ruff = { version = ">=0.0.267,<0.1.2", python = "^3.10" }
+ruff = { version = ">=0.3.2", python = "^3.10" }
 
 [tool.poetry.group.dev.dependencies]
 pytest = "*"
@@ -41,7 +41,7 @@ requests = "*"
 coverage = ">=6.5,<8.0"
 httpx = "*"
 tox = "^4.5.1"
-towncrier = "24.8.0"
+towncrier = "^22.12.0"
 
 [tool.poetry.group.distributing]
 optional = true
@@ -88,36 +88,6 @@ template = "changelog.d/changelog_template.jinja"
 title_format = "## [{version}](https://github.com/long2ice/fastapi-cache/tree/{version}) - {project_date}"
 issue_format = "[#{issue}](https://github.com/long2ice/fastapi-cache/issues/{issue})"
 
-[[tool.towncrier.type]]
-directory = 'feature'
-name = 'New Features'
-showcontent = true
-
-[[tool.towncrier.type]]
-directory = 'removed'
-name = 'Deprecated/Removed Features'
-showcontent = true
-
-[[tool.towncrier.type]]
-directory = 'fix'
-name = 'Bug Fixes'
-showcontent = true
-
-[[tool.towncrier.type]]
-directory = 'build'
-name = 'Build Changes'
-showcontent = true
-
-[[tool.towncrier.type]]
-directory = 'doc'
-name = 'Documentation'
-showcontent = true
-
-[[tool.towncrier.type]]
-directory = 'misc'
-name = 'Everything Else'
-showcontent = true
-
 [tool.pyright]
 strict = ["fastapi_cache", "tests"]
 pythonVersion = "3.8"
@@ -126,9 +96,9 @@ pythonVersion = "3.8"
 addopts = "-p no:warnings"
 
 [tool.ruff]
-ignore = ["E501"]
 line-length = 80
-select = [
+lint.ignore = ["E501"]
+lint.select = [
     "B", # flake8-bugbear
     "C4", # flake8-comprehensions
     "E", # pycodestyle errors

tests/pyproject.toml

@@ -1,10 +1,10 @@
 [tool.ruff]
 extend = "../pyproject.toml"
-extend-select = [
+lint.extend-select = [
     "PT", # flake8-pytest-style
 ]
-ignore = ["S101"]
+lint.ignore = ["S101"]
 
-[tool.ruff.isort]
+[tool.ruff.lint.isort]
 known-first-party = ["examples", "fastapi_cache"]
 

tests/test_codecs.py

@@ -1,5 +1,5 @@
 from dataclasses import dataclass
-from typing import Any, Optional, Type
+from typing import Any, Dict, List, Optional, Type
 
 import pytest
 from pydantic import BaseModel
@@ -41,9 +41,7 @@ def test_pickle_coder(value: Any) -> None:
     assert decoded_value == value
 
 
-# vicchi: 2025/01/17
-# test values and tests commented out until #460 is resolved due to removal
-# of support for decoding JSON to a custom type
+# vicchi: 2024/11/09 - some values commented out until #460 is resolved
 @pytest.mark.parametrize(
     ("value", "return_type"),
     [
@@ -52,8 +50,9 @@ def test_pickle_coder(value: Any) -> None:
         ("some_string", str),
         ("some_string", None),
         # ((1, 2), Tuple[int, int]),
+        ([1, 2, 3], List[int]),
         ([1, 2, 3], None),
-        # ({"some_key": 1, "other_key": 2}, None),
+        ({"some_key": 1, "other_key": 2}, Dict[str, int]),
         ({"some_key": 1, "other_key": 2}, None),
         # (DCItem(name="foo", price=42.0, description="some dataclass item", tax=0.2), DCItem),
         # (PDItem(name="foo", price=42.0, description="some pydantic item", tax=0.2), PDItem),
@@ -66,6 +65,7 @@ def test_json_coder(value: Any, return_type: Type[Any]) -> None:
     assert decoded_value == value
 
 
+# vicchi: 2024/11/09 - test commented out until #460 is resolved
 # def test_json_coder_validation_error() -> None:
 #     invalid = b'{"name": "incomplete"}'
 #     with pytest.raises(ValidationError):

tests/test_decorator.py

@@ -23,8 +23,6 @@ def test_datetime() -> None:
         response = client.get("/datetime")
         assert response.headers.get("X-FastAPI-Cache") == "MISS"
         now = response.json().get("now")
-        # now_ = pendulum.now()
-        # assert pendulum.parse(now) == now_
         now_ = pendulum.parse(now)
         response = client.get("/datetime")
         assert response.headers.get("X-FastAPI-Cache") == "HIT"
@@ -32,12 +30,10 @@ def test_datetime() -> None:
         assert pendulum.parse(now) == now_
         time.sleep(3)
         response = client.get("/datetime")
-        # now = response.json().get("now")
         assert response.headers.get("X-FastAPI-Cache") == "MISS"
         now = response.json().get("now")
         now = pendulum.parse(now)
         assert now != now_
-        # assert now == pendulum.now()
 
 
 def test_date() -> None:

tox.ini

@@ -30,7 +30,7 @@ skip_install = true
 commands_pre =
     poetry install --no-root --with=linting --sync --all-extras
 commands =
-    ruff check --show-source .
+    ruff check .
     mypy
     pyright