Yaniv Bronhaim has submitted this change and it was merged.

Change subject: ssl: accept_ssl needs to close socket
......................................................................


ssl: accept_ssl needs to close socket

When we connect using plain connection timeout of 3 seconds occurs during
ssl handshake. In newer code we close a socket and disconnect the client.
This is not the case for this code so we had to add closing connection to
disconnect the client.

Change-Id: I2e521db78bf35334ad3bbfafd8396401924d254f
Signed-off-by: pkliczew <[email protected]>
Bug-Url: https://bugzilla.redhat.com/1148772
Reviewed-on: http://gerrit.ovirt.org/35227
Tested-by: Piotr Kliczewski <[email protected]>
Reviewed-by: Oved Ourfali <[email protected]>
Reviewed-by: Yaniv Bronhaim <[email protected]>
---
M lib/vdsm/SecureXMLRPCServer.py
1 file changed, 3 insertions(+), 0 deletions(-)

Approvals:
  Piotr Kliczewski: Verified
  Yaniv Bronhaim: Looks good to me, approved
  Oved Ourfali: Looks good to me, but someone else must approve



-- 
To view, visit http://gerrit.ovirt.org/35227
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I2e521db78bf35334ad3bbfafd8396401924d254f
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Piotr Kliczewski <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to