Alon Bar-Lev has posted comments on this change.

Change subject: engine: Method to fetch SSH fingerprint of server
......................................................................


Patch Set 3:

Still this method is not optimal.

1. You already have the server key in serverKey after connection, no need to 
extract this as you did.

2. There is no reason to disconnect if user approves.

Either a callback is appropriate or connect can have two phases, connectOnly 
and authenticate.

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

Gerrit-MessageType: comment
Gerrit-Change-Id: I398fac6cbf641b49c8281937280ae0c351e1d3b6
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Shireesh Anjal <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Dhandapani Gopal <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Omer Frenkel <[email protected]>
Gerrit-Reviewer: Shireesh Anjal <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to