Great, Thank you all! On Thu, Aug 11, 2022 at 7:38 PM Ilya Maximets <i.maxim...@ovn.org> wrote:
> 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
