-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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.
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
Implement high integrity mode for commands #2741
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Went through on call - notes inline
src/StackExchange.Redis/Configuration/DefaultOptionsProvider.cs
Outdated
Show resolved
Hide resolved
src/StackExchange.Redis/Configuration/DefaultOptionsProvider.cs
Outdated
Show resolved
Hide resolved
- add transaction work to basic tests, to ensure handled
- burn the connection on failure - add initial metrics
@@ -6,7 +6,7 @@ | |||
<AssemblyName>BasicTest</AssemblyName> | |||
<OutputType>Exe</OutputType> | |||
<PackageId>BasicTest</PackageId> | |||
<RuntimeIdentifiers>win7-x64</RuntimeIdentifiers> | |||
<!--<RuntimeIdentifiers>win7-x64</RuntimeIdentifiers>--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uncomment or remove?
@@ -6,7 +6,7 @@ | |||
<AssemblyName>BasicTestBaseline</AssemblyName> | |||
<OutputType>Exe</OutputType> | |||
<PackageId>BasicTestBaseline</PackageId> | |||
<RuntimeIdentifiers>win7-x64</RuntimeIdentifiers> | |||
<!--<RuntimeIdentifiers>win7-x64</RuntimeIdentifiers>--> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Uncomment or remove?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it isn't the core - I'm fine with leaving it for context
Fixes #2706
This adds additional
ECHO
items after every outbound request, and adds validation of that correlation marker before treating responses as valid.This is initial draft only; needs conversion to sequence and testing of failure modes.
Discussion points