TarsCloud / TarsFramework

Tars Basic service framework
BSD 3-Clause "New" or "Revised" License
200 stars 143 forks source link

NodeServer在windows平台上会出现注册空节点的问题 #81

Closed flamefox closed 3 years ago

flamefox commented 3 years ago

更新node节点的ip在NodeServer::onUpdateConfig中实现 而上报是在KeepAliveThread中上报的 但是不知道为啥一些情况会出现下面的log

2020-12-16 14:29:16|4672|DEBUG|keepalivethread.cpp:KeepAliveThread::registerNode:149|registerNode begin===============|node name|
2020-12-16 14:29:17|4672|DEBUG|keepalivethread.cpp:KeepAliveThread::registerNode:156|register node succ

所以NODEID的更新是不是有问题,造成注册名为空的情况出现,现在暂时做法是调整了一下注册的逻辑 如果NodeID为空则不进行注册操作

操作系统:win764 framework版本: Revision: a589afad7aec6dfdc2ed20b290e57d48db87e2ce

ruanshudong commented 3 years ago

这个能提交一个pr?

ruanshudong commented 3 years ago

貌似代码没看出来问题, ServerConfig::LocalIp是启动时初始化好的, 没道理为空, 奇怪了