Sentinel: Add HELLO support

This allows connections through Sentinel via RESP3 by adding `HELLO` to the command map.
StackExchange · Nov 23, 2023 · e46d8a7 · e46d8a7
1 parent 79e8346
commit e46d8a7
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/src/StackExchange.Redis/CommandMap.cs b/src/StackExchange.Redis/CommandMap.cs
@@ -93,7 +93,7 @@ public sealed class CommandMap
         /// </summary>
         /// <remarks><seealso href="https://redis.io/topics/sentinel"/></remarks>
         public static CommandMap Sentinel { get; } = Create(new HashSet<string> {
-            "auth", "ping", "info", "role", "sentinel", "subscribe", "shutdown", "psubscribe", "unsubscribe", "punsubscribe" }, true);
+            "auth", "hello", "ping", "info", "role", "sentinel", "subscribe", "shutdown", "psubscribe", "unsubscribe", "punsubscribe" }, true);
 
         /// <summary>
         /// Create a new <see cref="CommandMap"/>, customizing some commands.

diff --git a/tests/StackExchange.Redis.Tests/SentinelTests.cs b/tests/StackExchange.Redis.Tests/SentinelTests.cs
@@ -86,6 +86,7 @@ public async Task PrimaryConnectAsyncTest()
     }
 
     [Fact]
+    [RunPerProtocol]
     public void SentinelConnectTest()
     {
         var options = ServiceOptions.Clone();
-Original file line number
+Diff line change
@@ Expand Up / @@ -86,6 +86,7 @@ public async Task PrimaryConnectAsyncTest() @@
         }
         [Fact]
+        [RunPerProtocol]
         public void SentinelConnectTest()
         {
             var options = ServiceOptions.Clone();
@@ Expand Down @@