372046933 commented on code in PR #3179:
URL: https://github.com/apache/brpc/pull/3179#discussion_r2660352579


##########
src/brpc/socket.cpp:
##########
@@ -1296,7 +1297,14 @@ int Socket::Connect(const timespec* abstime,
     CHECK_EQ(0, butil::make_close_on_exec(sockfd));
     // We need to do async connect (to manage the timeout by ourselves).
     CHECK_EQ(0, butil::make_non_blocking(sockfd));
-    
+    if (!_device_name.empty()) {

Review Comment:
   不是很确定 https://github.com/apache/brpc/issues/3157 需要的是指定source 
IP,还是指定网络设备。如果只需要指定source IP, 之前的bind可以满足需求。这里要不要把之前的那个client_host选项一并加进来?



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to