Skip to content
This repository was archived by the owner on Dec 24, 2022. It is now read-only.

Commit 126e68b

Browse files
committed
Change RedisClient and RedisClient Factories to accept a rich RedisEndpoint instead
1 parent 59d9e7b commit 126e68b

6 files changed

Lines changed: 20 additions & 16 deletions

File tree

src/ServiceStack.Redis/IRedisClientFactory.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,6 @@ namespace ServiceStack.Redis
1616
{
1717
public interface IRedisClientFactory
1818
{
19-
RedisClient CreateRedisClient(string host, int port);
19+
RedisClient CreateRedisClient(RedisEndpoint config);
2020
}
2121
}

src/ServiceStack.Redis/RedisCacheClientFactory.cs

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ public class RedisCacheClientFactory : IRedisClientFactory
2222
{
2323
public static RedisCacheClientFactory Instance = new RedisCacheClientFactory();
2424

25-
public RedisClient CreateRedisClient(string host, int port)
26-
{
27-
return new RedisClient(host, port);
28-
}
29-
}
25+
public RedisClient CreateRedisClient(RedisEndpoint config)
26+
{
27+
return new RedisClient(config);
28+
}
29+
}
3030
}

src/ServiceStack.Redis/RedisClientFactory.cs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ public class RedisClientFactory : IRedisClientFactory
2222
{
2323
public static RedisClientFactory Instance = new RedisClientFactory();
2424

25-
public RedisClient CreateRedisClient(string host, int port)
26-
{
27-
return new RedisClient(host, port);
28-
}
25+
public RedisClient CreateRedisClient(RedisEndpoint config)
26+
{
27+
return new RedisClient(config);
28+
}
2929
}
3030
}

src/ServiceStack.Redis/Support/Queue/Implementation/SerializingRedisClient.cs

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,15 @@ namespace ServiceStack.Redis.Support.Queue.Implementation
77
public class SerializingRedisClient : RedisClient
88
{
99
private ISerializer serializer = new ObjectSerializer();
10+
11+
public SerializingRedisClient(string host)
12+
: base(host) {}
13+
14+
public SerializingRedisClient(RedisEndpoint config)
15+
: base(config) {}
1016

1117
public SerializingRedisClient(string host, int port)
12-
: base(host, port)
13-
{
14-
}
18+
: base(host, port) {}
1519

1620
/// <summary>
1721
/// customize the client serializer

src/ServiceStack.Redis/Support/Queue/Implementation/SerializingRedisClientFactory.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,9 @@ public class SerializingRedisClientFactory : IRedisClientFactory
77
{
88
public static SerializingRedisClientFactory Instance = new SerializingRedisClientFactory();
99

10-
public RedisClient CreateRedisClient(string host, int port)
10+
public RedisClient CreateRedisClient(RedisEndpoint config)
1111
{
12-
return new SerializingRedisClient(host, port);
12+
return new SerializingRedisClient(config);
1313
}
1414
}
1515
}

tests/ServiceStack.Redis.Tests/PooledRedisClientManagerTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ private void SetupRedisFactoryMock()
3737
{
3838
mockFactory = new Mock<IRedisClientFactory>();
3939
mockFactory.Expect(x => x.CreateRedisClient(
40-
It.IsAny<string>(), It.IsAny<int>()))
40+
It.IsAny<RedisEndpoint>()))
4141
.Returns((Func<string, int, RedisClient>)((host, port) => new RedisClient(host, port)));
4242
}
4343

0 commit comments

Comments
 (0)