Skip to content

Commit 429d588

Browse files
ref: remove incr_version / decr_version (#15)
these methods are unsafe with our usage of cache
1 parent 89794d3 commit 429d588

File tree

2 files changed

+10
-4
lines changed

2 files changed

+10
-4
lines changed

django-stubs/core/cache/backends/base.pyi

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -60,10 +60,6 @@ class BaseCache:
6060
async def adelete_many(self, keys: Iterable[Any], version: Any | None = None) -> None: ...
6161
def clear(self) -> None: ...
6262
async def aclear(self) -> None: ...
63-
def incr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
64-
async def aincr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
65-
def decr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
66-
async def adecr_version(self, key: Any, delta: int = 1, version: Any | None = None) -> int: ...
6763
def close(self, **kwargs: Any) -> None: ...
6864
async def aclose(self, **kwargs: Any) -> None: ...
6965

scripts/stubtest/allowlist.txt

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -480,3 +480,13 @@ django.contrib.auth.views.UserModel
480480
# Using the definitions from `dict` (from typeshed)
481481
django.contrib.sessions.backends.base.SessionBase.get
482482
django.contrib.sessions.backends.base.SessionBase.setdefault
483+
484+
# sentry: we remove these methods because they are unsafe for our usage
485+
django.core.cache.BaseCache.adecr_version
486+
django.core.cache.BaseCache.aincr_version
487+
django.core.cache.BaseCache.decr_version
488+
django.core.cache.BaseCache.incr_version
489+
django.core.cache.backends.base.BaseCache.adecr_version
490+
django.core.cache.backends.base.BaseCache.aincr_version
491+
django.core.cache.backends.base.BaseCache.decr_version
492+
django.core.cache.backends.base.BaseCache.incr_version

0 commit comments

Comments
 (0)