old-bear commented on issue #1096: URL: https://github.com/apache/incubator-brpc/issues/1096#issuecomment-620589519
感谢指正，我看了下 DoRead中确实存在bug，但具体问题好像与你描述的不同： 1. `append_from_SSL_channel`中不断调用`SSL_read`，已经读入了一些内容，`nr>0`，此时最后一次`SSL_read`返回`rc=0`，而`append_from_SSL_channel`返回`nr (>0)` 2. `DoRead`中进入default分支，打印了`Fail to read from ssl_fd=id`，返回`nr (>0)` 3. `InputMessenger`中调用`DoRead`得到`nr>0`，处理消息，然后循环`while (!read_eof)`，继续调用`DoRead -> SSL_read`，此时则rc=0，一路return出去，被当做eof 因此，我这边看这个bug主要在于会错误打印一句warning日志，应该不影响fd关闭或者fd泄漏 如果你这边有具体复现场景描述，或者相关日志，希望能提供一下，谢谢 另外，SSL握手目前被视作一种**连通性**指标，如果server拒绝与client建立SSL（无论何种因素），那么后续任何请求肯定无法成功。 ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
