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
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