Skip to content

Conversation

@asottile-sentry
Copy link

these methods are unsafe with our usage of cache

we use str for cache version as a hashed value of model fields -- these methods manipulate version as an int -- fortunately we never use them

@asottile-sentry asottile-sentry requested review from a team June 14, 2024 13:43
these methods are unsafe with our usage of cache
@asottile-sentry asottile-sentry merged commit ac84848 into sentry-5.0.2 Jun 14, 2024
@asottile-sentry asottile-sentry deleted the remove-incr-decr branch June 14, 2024 18:01
asottile-sentry added a commit that referenced this pull request Jul 31, 2024
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Sep 24, 2024
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Oct 28, 2024
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Jan 14, 2025
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Feb 7, 2025
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Apr 28, 2025
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Jun 17, 2025
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Jul 21, 2025
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Sep 12, 2025
these methods are unsafe with our usage of cache
asottile-sentry added a commit that referenced this pull request Sep 15, 2025
asottile-sentry added a commit that referenced this pull request Sep 15, 2025
* Revert "ref: remove incr_version / decr_version (#15)"

This reverts commit 1020f9a.

* Revert "cache's version can be Any (#14)"

This reverts commit fc8ff33.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants