Handle nil value in GetbyteForSSLSocket::getbyte#306
Handle nil value in GetbyteForSSLSocket::getbyte#306HarlemSquirrel merged 1 commit intoruby-ldap:masterfrom
Conversation
|
I'm not sure that this will completely fix #266. This might just result in a different error. But it seems to me that this should be the correct behavior, and it looks like at least some of the places that call this do expect to get nil back sometimes. |
|
thanks for the PR! Currently I am not able to push a new release, because I am on vacation an fortunately far away from my computer. I am back at the beginning of August. |
|
Any chances for merging this pull request in? |
|
Any progress on this? |
|
This PR fixed an issue for us when interfacing with the Duo Authentication LDAP proxy. It seems to be low risk and would be helpful if merged. |
|
I also ran into this bug when using haproxy as LDAPS proxy in front of a pool of AD LDAP servers and haproxy dropped the connection because none of the backendas where healthy. |
|
@HarlemSquirrel Would it be possible to get a review of this fix please? |
Related to #266