Re: [libvirt] [libvirt PATCH] qemu: Add support for specifying SPICE TLS ciphers
On Tue, Apr 03, 2018 at 08:11:05PM +0200, Jiri Denemark wrote:
On Tue, Apr 03, 2018 at 17:23:50 +0200, Ján Tomko wrote:
> From: Christophe Fergeau <cfergeau(a)redhat.com>
>
> This commit adds a 'spice_tls_ciphers' parameter in
> qemu.conf which allows to configure which TLS ciphers
> SPICE will be using for its TLS connections.
>
> https://bugzilla.redhat.com/show_bug.cgi?id=1562032
>
> Signed-off-by: Christophe Fergeau <cfergeau(a)redhat.com>
> Signed-off-by: Ján Tomko <jtomko(a)redhat.com>
> ---
> This is mostly useful as a workaround for missing crypto policies,
> so I'm not sure if it's upstream material.
Are OpenSSL crypto policies supported upstream? If so, I think we should
just rely on them and leave this workaround for downstreams if they want
it. Also, what would we do if spice changed its TLS code to use another
library, wouldn't it force us to translate the parameters from OpenSSL
to the other library if this happens and this code is merged upstream?
The latter is actually my biggest concern with this. I would really like
to get SPICE to use the QEMU TLS creds framework, so we can do the normal
-object tls-creds-x509 ... setup as we do for other parts ofo QEMU. This
would mean SPICE using GNUTLS format for specifying ciphers. In fact the
GNUTLS format specifies ciphers and TLS protocol versions - both of which
the quoted bug is asking for - whereas this patch only specifies ciphers
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|