-
-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Try to fix issue #599 with an untested patch #600
Conversation
a110332
to
03e6173
Compare
So looking at SSL_R_SHORT_READ it was only used in SSLv2, and it seems related to requesting certificates. I suggest you just remove the whole if and just: |
03e6173
to
dcfac07
Compare
updated thanks! |
Hmm, I might have suggested to remove too much, maybe you do want to return that it's a tls error. |
d89f9b3
to
1dd0711
Compare
this way? |
Yes, looks good. |
Would it be possible to get this patch in? It's currently blocking me as well |
A fix for this in theory has been committed to develop branch. |
closing thanks! |
This might fix issue #599, but I have no clues about what does it mean