Fix sentinel default port when connecting with configuration string by ejsmith · Pull Request #1525 · StackExchange/StackExchange.Redis

Expand Up @@ -840,7 +840,7 @@ internal ServerEndPoint AnyConnected(ServerType serverType, uint startOffset, Re public static Task<ConnectionMultiplexer> ConnectAsync(string configuration, TextWriter log = null) { SocketConnection.AssertDependencies(); return ConnectAsync(PrepareConfig(configuration), log); return ConnectAsync(ConfigurationOptions.Parse(configuration), log); }
private static async Task<ConnectionMultiplexer> ConnectImplAsync(ConfigurationOptions configuration, TextWriter log = null) Expand Down Expand Up @@ -918,12 +918,7 @@ internal static ConfigurationOptions PrepareConfig(object configuration, bool se
if (sentinel) { // this is required when connecting to sentinel servers config.TieBreaker = ""; config.CommandMap = CommandMap.Sentinel;
// use default sentinel port config.EndPoints.SetDefaultPorts(26379); config.SetSentinelDefaults();
return config; } Expand Down Expand Up @@ -1017,7 +1012,7 @@ private static ConnectionMultiplexer CreateMultiplexer(ConfigurationOptions conf /// <param name="log">The <see cref="TextWriter"/> to log to.</param> public static ConnectionMultiplexer Connect(string configuration, TextWriter log = null) { return Connect(PrepareConfig(configuration), log); return Connect(ConfigurationOptions.Parse(configuration), log); }
/// <summary> Expand Down