From Yaniv Bronhaim <ybron...@redhat.com>: Yaniv Bronhaim has submitted this change and it was merged. ( https://gerrit.ovirt.org/80373 )
Change subject: ssl: handle handshake errors ...................................................................... ssl: handle handshake errors A previous patch that handled the ssl hanshake errors was merged without updating log entries for ssl errors. We log now only the occurence of the error and the offending IP address that caused the error. We fix the issue by handling both errors and cleanly closing file descriptor. Change-Id: I2cc1d249b57d9893b3bc35dcf5d72674a83edf36 Bug-url: https://bugzilla.redhat.com/1473295 Signed-off-by: Piotr Kliczewski <piotr.kliczew...@gmail.com> Signed-off-by: Irit Goihman <igoih...@redhat.com> --- M lib/vdsm/sslutils.py 1 file changed, 4 insertions(+), 0 deletions(-) Approvals: Martin Peřina: Looks good to me, but someone else must approve Yaniv Bronhaim: Looks good to me, approved Jenkins CI: Passed CI tests Irit Goihman: Verified -- To view, visit https://gerrit.ovirt.org/80373 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: merged Gerrit-Change-Id: I2cc1d249b57d9893b3bc35dcf5d72674a83edf36 Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: ovirt-4.1 Gerrit-Owner: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com> Gerrit-Reviewer: Francesco Romani <from...@redhat.com> Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Martin Peřina <mper...@redhat.com> Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczew...@gmail.com> Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com> Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org> _______________________________________________ vdsm-patches mailing list -- vdsm-patches@lists.fedorahosted.org To unsubscribe send an email to vdsm-patches-le...@lists.fedorahosted.org