Skip to content

Bump StackExchange.Redis from 3.0.25 to 3.1.0 #541

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
dependabot wants to merge 1 commit into
base: master
Choose a base branch
from
+1 −1
Open

Bump StackExchange.Redis from 3.0.25 to 3.1.0 #541

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion Directory.Packages.props
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
<!-- primary library -->
<PackageVersion Include="NetTopologySuite" Version="2.6.0" />
<PackageVersion Include="System.Text.Json" Version="10.0.10" />
<PackageVersion Include="StackExchange.Redis" Version="3.0.25" />
<PackageVersion Include="StackExchange.Redis" Version="3.1.0" />

@chatgpt-codex-connector chatgpt-codex-connector Bot Jul 31, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Keep the public StackExchange.Redis floor at 3.0.x

This central version is consumed by src/NRedisStack/NRedisStack.csproj, so packages built from the library now require StackExchange.Redis >= 3.1.0 instead of keeping the documented 3.0.x compatibility floor. The retry-category code explicitly avoids referencing the 3.1.0-only enum names so older 3.0.x consumers still work (src/NRedisStack/RedisStackCommands/CommandCategories.cs:11-13), and the tests already use VersionOverride="3.1.0" specifically to check those names while staying ahead of the library floor (tests/NRedisStack.Tests/NRedisStack.Tests.csproj:39-42). Consumers that remain on StackExchange.Redis 3.0.x therefore lose compatibility without a code need for the library itself to raise the floor.

Useful? React with 👍 / 👎.

@chatgpt-codex-connector chatgpt-codex-connector Bot Jul 31, 2026

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Avoid casting retry-wrapped async databases to IDatabase

With StackExchange.Redis 3.1.0, callers can pass a retry-wrapped database from WithRetry() to NRedisStack's async command classes because it is an IDatabaseAsync, but that wrapper is async-only and does not implement IDatabase. Every async module call routes through Auxiliary.ExecuteAsync, which unconditionally casts ((IDatabase)db).SetInfoInPipeline() (src/NRedisStack/Auxiliary.cs:75-77), so new JsonCommandsAsync(db.WithRetry()).GetAsync(...) throws InvalidCastException before the command is sent and the new retry categorization is unusable for async NRedisStack commands.

Useful? React with 👍 / 👎.

<!-- tests, etc -->
<PackageVersion Include="BouncyCastle.Cryptography" Version="2.7.0" />
<PackageVersion Include="coverlet.collector" Version="10.0.1" />
Expand Down
Loading