This modules doesn't grok LDAP at all.
Most importantly, it doesn't understand that LDAP is a pipelining protocol: It allows multiple requests to be ongoing at the same time, and allowing the server to reply to them in any order it sees fit (including mixed order replies). This synchronization it does makes it hard to make a responsive and scalable server (irony points for nonetheless having an abandon method, as if that could do anything in this design).
It's lack of abstraction around the request/response, and the (ab)use of inheritance are unfortunate, though less problematic on the long run.
3 out of 3 found this review helpful. Was this review helpful to you? Yes No