Yair Zaslavsky has posted comments on this change.

Change subject: Wrap validation of fingerprint in each connect using 
EngineSSHClient
......................................................................


Patch Set 8: (2 inline comments)

....................................................
File 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/VdsDeploy.java
Line 838:      */
Line 839:     public void execute(AuthenticationMethod auth) throws Exception {
Line 840:         InputStream in = null;
Line 841:         try {
Line 842:             _dialog.connect(_vds);
where is disconnect performed?
Line 843:             _messages.post(
Line 844:                 InstallerMessages.Severity.INFO,
Line 845:                 String.format(
Line 846:                     "Connected to host %1$s with SSH key fingerprint: 
%2$s",


....................................................
File 
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/crypt/OpenSSHUtils.java
Line 137:      * @return an array of bytes containing the fingerprint of the key
Line 138:      */
Line 139:     public static final byte[] getKeyFingerprintBytes(final PublicKey 
key) {
Line 140:         if (key == null) {
Line 141:             log.error("Public key is uninitialized, failed to 
retreive fingerprint.");
s/uninitilized/null
Line 142:             return null;
Line 143:         }
Line 144: 
Line 145:         // Get the serialized version of the key:


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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ic01517a153406c8bafc672c20b0bf8686763a2f5
Gerrit-PatchSet: 8
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Sahina Bose <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to