On Thu, Oct 21, 2021 at 12:01:35PM +0200, Stefan Reiter wrote: > VNC only supports 'keep' here, enforce this via a seperate > SetPasswordActionVnc enum and mark the option 'deprecated' (as it is > useless with only one value possible). > > Also add a deprecation note to docs.
IMHO we should just implement 'fail' and 'disconnect' in the VNC server. Consistency across SPICE and VNC is more user friendly than restricting the available options, and all three policies are useful behaviours for VNC. So I'm not in favour of adding this deprecation. 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 :|
