ruby-ldap/ruby-net-ldap — pull request #144 by mtodd (2014-10-22 08:34:10+00:00)
Title: Queue reads everywhere

After #138 was merged, I realized that *any* reads from the socket are susceptible from the same problem as described in #136.

I've added a reproduction (as an integration test) to illustrate the point, and also includes a trivial fix based on replacing `read` usage with `queued_read`.

This needs to be done for any usage of `read` in `Net::LDAP::Connection`:

- [ ] `setup_encryption`
- [ ] `bind_simple`
- [ ] `bind_sasl`
- [ ] `modify`
- [x] `add`
- [ ] `rename`
- [ ] `delete`

cc @jch @schaary