[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PULL 10/10] monitor: add vnc websockets
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PULL 10/10] monitor: add vnc websockets |
Date: |
Fri, 19 Dec 2014 14:06:47 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Copying Eric again, even though the QAPI schema change here is pretty
trivial.
Gerd Hoffmann <address@hidden> writes:
> Add websockets bool to VncBasicInfo, report websocket server sockets,
> flag websocket client connections.
>
> Signed-off-by: Gerd Hoffmann <address@hidden>
> ---
> qapi-schema.json | 5 ++++-
> ui/vnc.c | 15 ++++++++++++---
> 2 files changed, 16 insertions(+), 4 deletions(-)
>
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 2d45d4c..07deb71 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -672,12 +672,15 @@
> #
> # @family: address family
> #
> +# @websocket: true in case the socket is a websocket (since 2.3).
> +#
> # Since: 2.1
> ##
> { 'type': 'VncBasicInfo',
> 'data': { 'host': 'str',
> 'service': 'str',
> - 'family': 'NetworkAddressFamily' } }
> + 'family': 'NetworkAddressFamily',
> + 'websocket': 'bool' } }
>
> ##
> # @VncServerInfo
> diff --git a/ui/vnc.c b/ui/vnc.c
> index e730059..fb8068f 100644
> --- a/ui/vnc.c
> +++ b/ui/vnc.c
> @@ -353,6 +353,9 @@ static VncClientInfo *qmp_query_vnc_client(const VncState
> *client)
> info->base->host = g_strdup(host);
> info->base->service = g_strdup(serv);
> info->base->family = inet_netfamily(sa.ss_family);
> +#ifdef CONFIG_VNC_WS
> + info->base->websocket = client->websocket;
> +#endif
>
> #ifdef CONFIG_VNC_TLS
> if (client->tls.session && client->tls.dname) {
> @@ -457,6 +460,7 @@ out_error:
> }
>
> static VncBasicInfoList *qmp_query_server_entry(int socket,
> + bool websocket,
> VncBasicInfoList *prev)
> {
> VncBasicInfoList *list;
> @@ -477,6 +481,7 @@ static VncBasicInfoList *qmp_query_server_entry(int
> socket,
> info->host = g_strdup(host);
> info->service = g_strdup(serv);
> info->family = inet_netfamily(sa.ss_family);
> + info->websocket = websocket;
>
> list = g_new0(VncBasicInfoList, 1);
> list->value = info;
> @@ -572,12 +577,13 @@ VncInfo2List *qmp_query_vnc2(Error **errp)
> info->display = g_strdup(dev->id);
> }
> if (vd->lsock != -1) {
> - info->server = qmp_query_server_entry(vd->lsock,
> + info->server = qmp_query_server_entry(vd->lsock, false,
> info->server);
> }
> #ifdef CONFIG_VNC_WS
> if (vd->lwebsock != -1) {
> - /* TODO */
> + info->server = qmp_query_server_entry(vd->lwebsock, true,
> + info->server);
> }
> #endif
>
> @@ -3304,10 +3310,13 @@ void vnc_display_open(const char *id, Error **errp)
> {
> VncDisplay *vs = vnc_display_find(id);
> QemuOpts *opts = qemu_opts_find(&qemu_vnc_opts, id);
> - const char *display, *websocket, *share, *device_id;
> + const char *display, *share, *device_id;
> QemuConsole *con;
> int password = 0;
> int reverse = 0;
> +#ifdef CONFIG_VNC_WS
> + const char *websocket;
> +#endif
> #ifdef CONFIG_VNC_TLS
> int tls = 0, x509 = 0;
> const char *path;
Reviewed-by: Markus Armbruster <address@hidden>
- [Qemu-devel] [PULL 00/10] vnc: add support for multiple vnc displays, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 02/10] vnc: remove unused DisplayState parameter, add id instead., Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 06/10] vnc: update docs/multiseat.txt, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 04/10] vnc: switch to QemuOpts, allow multiple servers, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 09/10] monitor: add query-vnc2 command, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 10/10] monitor: add vnc websockets, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 03/10] vnc: add display id to acl names, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 05/10] vnc: allow binding servers to qemu consoles, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 07/10] vnc: track & limit connections, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 08/10] vnc: factor out qmp_query_client_list, Gerd Hoffmann, 2014/12/19
- [Qemu-devel] [PULL 01/10] vnc: remove vnc_display global, Gerd Hoffmann, 2014/12/19
- Re: [Qemu-devel] [PULL 00/10] vnc: add support for multiple vnc displays, Markus Armbruster, 2014/12/19