New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
SSLSocket.sendall() does not return None on success like socket.sendall() #70139
Comments
socket.sendall() returns None if it succeeded <https://docs.python.org/3/library/socket.html#socket.socket.sendall\> However, SSLSocket.sendall() returns the amount of bytes written on success. |
Changed SSLSocket.sendall() to return None. Also added a check of the return value of the SSLSocket.send(), SSLSocket.sendall() in the tests. |
Agreed that the documentation and implementation should match. But this seems like a rather low-priority bug. What use case relies on the return value being None? If there is no immediate need for this change, it might be safer to just make it in 3.6, to minimize compatibility problems. But if it is fixed in 3.5 it should also be fixed in 2.7. I left some suggestions to simplify the code. |
Thanks for the review. |
Thanks for your work Aviv. This version looks good to me. |
New changeset 92947704321c by Martin Panter in branch 'default': |
I made some simple tweaks to avoid long lines. |
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
bugs.python.org fields:
The text was updated successfully, but these errors were encountered: