On 8/10/22 11:37, Timothy Redaelli wrote: > On Mon, 8 Aug 2022 12:32:42 -0500 > Miro Tomaska <mtoma...@redhat.com> wrote: > >> Python std library SSLSocket.send does not allow non-zero value for the >> optional flag. >> >> pyOpenSSL was recently switched for the Python standard library ssl module >> commit 68543dd523bd00f53fa7b91777b962ccb22ce679 (python: Replace pyOpenSSL >> with ssl). >> Python SSLsocket.send() does not allow non-zero optional flag and it will >> explicitly >> raise an exception for that. pyOpenSSL did not nothing with this flag but >> kept >> it to be compatible with socket API. >> https://github.com/pyca/pyopenssl/blob/main/src/OpenSSL/SSL.py#L1844 >> >> Signed-off-by: Miro Tomaska <mtoma...@redhat.com> >> Reported-at: https://bugzilla.redhat.com/2115035 >> --- >> python/ovs/socket_util.py | 12 +++++++++++- >> 1 file changed, 11 insertions(+), 1 deletion(-) > > Thank you, I missed that one when I ported Python stuff from the > deprecated pyOpenSSL to ssl module and unluckily no tests detected this > problem, since it's only executed in case of socket errors > > Acked-By: Timothy Redaelli <tredae...@redhat.com>
Thanks! I added the still missing 'Fixes' tag and applied the change. Backported down to 2.17. Best regards, Ilya Maximets. _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev
