Conversation
On PyPy 3.9 `sock.close()` sometimes fails with `Bad file descriptor`, because the fd is already closed. Silence the exception. The problem is not a bug in python-ldap. Really use PyPy 3.9 in `tox.ini`. Signed-off-by: Christian Heimes <cheimes@redhat.com>
|
-1 on this: we should really figure out why the socket was closed already to an authoritative reason. Being opportunistic and then masking the error risks hiding hard to diagnose high severity bugs which we've had a hard time with using heimdal etc. in the past (a library closes its file descriptor, the fd # is reused by independent code and the library closes that one too). |
|
Be my guest and good luck figuring it out... The problem only occurs with PyPy 3.9 on GitHub Actions and only every few test runs. It's rather random. I have not been able to reproduce the issue locally either. In the mean time, I can work around the issue by restarting the failed test manually. |
|
Ok, I've had a try and can reproduce this quite reliably (even with |
On PyPy 3.9
sock.close()sometimes fails withBad file descriptor, because the fd is already closed. Silence the exception. The problem is not a bug in python-ldap.Really use PyPy 3.9 in
tox.ini.