-
Notifications
You must be signed in to change notification settings - Fork 113
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
将SCS升级到.net standard 2.0 #39
Conversation
…sServices.Service.ScsServiceApplication的_serviceClients)一直缓存着已经断开连接的客户端,最后耗光内存。 此问题是我在配合consul做服务发现时发现的,consul的tcp健康检查会不断的连接断开,在ReceiveCallback函数报异常去执行Disconnect函数,但是Disconnect函数因为条件判断为true直接return了
I fixed a bug about Disconnect on commit 8bf6fa8 |
Hello sir, I am tying to test your PR, but I am getting error, I am currently testing on DotNet Core 2.2
My code var endPoint = new ScsTcpEndPoint("localhost", 20000);
client = ScsServiceClientBuilder.CreateClient<IRemoteService>(endPoint); /*<-- error line*/
[ScsService]
public interface IRemoteService {
string Ping(string message);
string Login(string username, string password);
} |
DotNet Core uses Scs.Core instead of Scs |
Thanks for your implementation. I am late to merge, but better than never :) I will test it, then publish the package. |
@hikalkan Why don't you want to update the new version in Nuget . They are compatible with framework version. I tested it and it seems like it works well. |
No description provided.