Yaniv Bronhaim has posted comments on this change.
Change subject: Wrap validation of fingerprint in each connect using
EngineSSHClient
......................................................................
Patch Set 3: (1 inline comment)
....................................................
File
backend/manager/modules/utils/src/main/java/org/ovirt/engine/core/utils/ssh/EngineSSHClient.java
Line 50: super.connect();
Line 51: if (_vds != null) {
Line 52: String actual = getHostFingerprint();
Line 53: String expected = _vds.getSSHKeyFingerprint();
Line 54: if (!actual.equals(expected)) {
I failed to do that, glusterUtil uses EngineSSHClient, and doesn't depend on
bll. All mixes, we can move getFingerprint to SSHClient, then glusterUtil
doesn't need EngineSSHClient. but this also against what you asked. so what do
you suggest? moving to common?
Line 55: throw new GeneralSecurityException(
Line 56: String.format(
Line 57: "Invalid fingerprint %s, expected %s",
Line 58: actual,
--
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: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzasl...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: oVirt Jenkins CI Server
_______________________________________________
Engine-patches mailing list
Engine-patches@ovirt.org
http://lists.ovirt.org/mailman/listinfo/engine-patches
