Alon Bar-Lev has posted comments on this change. Change subject: engine: Fix fingerprint presentation in case of non self signed certificate ......................................................................
Patch Set 1: I do not understand.... there is a chain CA->CA->CA->End we should present the information of the top CA that we get, we should not present information of any other certificate. this means that we have all the information required within its certinfo entry. even if we do not have all required information within its own certinfo entry the frontend gets a list of entries, and can consult the entire chain for information. so I do not understand why adding a new field of issuer fingerprint into the certinfo entry is required while you can use the previous in chain. -- To view, visit http://gerrit.ovirt.org/36936 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I2dfb1db6fbaa387812a1113b244cd0d6b05c0530 Gerrit-PatchSet: 1 Gerrit-Project: ovirt-engine Gerrit-Branch: master Gerrit-Owner: Yair Zaslavsky <yzasl...@redhat.com> Gerrit-Reviewer: Alon Bar-Lev <alo...@redhat.com> Gerrit-Reviewer: Oved Ourfali <oourf...@redhat.com> Gerrit-Reviewer: automat...@ovirt.org Gerrit-HasComments: No _______________________________________________ Engine-patches mailing list Engine-patches@ovirt.org http://lists.ovirt.org/mailman/listinfo/engine-patches
