Cover Letter Detail
Show a cover letter.
GET /api/covers/811758/?format=api
{ "id": 811758, "url": "http://patchwork.ozlabs.org/api/covers/811758/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170908173801.15205-1-brandon.carpenter@cypherpath.com/", "project": { "id": 14, "url": "http://patchwork.ozlabs.org/api/projects/14/?format=api", "name": "QEMU Development", "link_name": "qemu-devel", "list_id": "qemu-devel.nongnu.org", "list_email": "qemu-devel@nongnu.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20170908173801.15205-1-brandon.carpenter@cypherpath.com>", "list_archive_url": null, "date": "2017-09-08T17:37:55", "name": "[v2,0/6] Update websocket code to more fully support the RFC", "submitter": { "id": 72031, "url": "http://patchwork.ozlabs.org/api/people/72031/?format=api", "name": "Brandon Carpenter", "email": "brandon.carpenter@cypherpath.com" }, "mbox": "http://patchwork.ozlabs.org/project/qemu-devel/cover/20170908173801.15205-1-brandon.carpenter@cypherpath.com/mbox/", "series": [ { "id": 2262, "url": "http://patchwork.ozlabs.org/api/series/2262/?format=api", "web_url": "http://patchwork.ozlabs.org/project/qemu-devel/list/?series=2262", "date": "2017-09-08T17:37:56", "name": "Update websocket code to more fully support the RFC", "version": 2, "mbox": "http://patchwork.ozlabs.org/series/2262/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/811758/comments/", "headers": { "Return-Path": "<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>", "X-Original-To": "incoming@patchwork.ozlabs.org", "Delivered-To": "patchwork-incoming@bilbo.ozlabs.org", "Authentication-Results": [ "ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=nongnu.org\n\t(client-ip=2001:4830:134:3::11; helo=lists.gnu.org;\n\tenvelope-from=qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org;\n\treceiver=<UNKNOWN>)", "ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (1024-bit key;\n\tunprotected) header.d=cypherpath.com header.i=@cypherpath.com\n\theader.b=\"CuirEYdL\"; dkim-atps=neutral" ], "Received": [ "from lists.gnu.org (lists.gnu.org [IPv6:2001:4830:134:3::11])\n\t(using TLSv1 with cipher AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xplKC03w0z9sDB\n\tfor <incoming@patchwork.ozlabs.org>;\n\tSat, 9 Sep 2017 03:52:46 +1000 (AEST)", "from localhost ([::1]:46741 helo=lists.gnu.org)\n\tby lists.gnu.org with esmtp (Exim 4.71) (envelope-from\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>)\n\tid 1dqNS0-0003fY-JM\n\tfor incoming@patchwork.ozlabs.org; Fri, 08 Sep 2017 13:52:20 -0400", "from eggs.gnu.org ([2001:4830:134:3::10]:39447)\n\tby lists.gnu.org with esmtp (Exim 4.71)\n\t(envelope-from <brandon.carpenter@cypherpath.com>)\n\tid 1dqNEb-0007pn-No\n\tfor qemu-devel@nongnu.org; Fri, 08 Sep 2017 13:38:30 -0400", "from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)\n\t(envelope-from <brandon.carpenter@cypherpath.com>)\n\tid 1dqNEX-0007j4-QN\n\tfor qemu-devel@nongnu.org; Fri, 08 Sep 2017 13:38:29 -0400", "from mail-pf0-x236.google.com ([2607:f8b0:400e:c00::236]:34218)\n\tby eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16)\n\t(Exim 4.71) (envelope-from <brandon.carpenter@cypherpath.com>)\n\tid 1dqNEX-0007iZ-L1\n\tfor qemu-devel@nongnu.org; Fri, 08 Sep 2017 13:38:25 -0400", "by mail-pf0-x236.google.com with SMTP id e1so5550444pfk.1\n\tfor <qemu-devel@nongnu.org>; Fri, 08 Sep 2017 10:38:25 -0700 (PDT)", "from servo.cypherpath.com (68-113-0-218.static.knwc.wa.charter.com.\n\t[68.113.0.218]) by smtp.gmail.com with ESMTPSA id\n\tc30sm4556833pgn.33.2017.09.08.10.38.23\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tFri, 08 Sep 2017 10:38:23 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=cypherpath.com; s=google;\n\th=mime-version:from:to:cc:subject:date:message-id:in-reply-to\n\t:references; bh=mCj3a3yGUZ0h5flR7J3TqswOq1XpSH8CvgOqnlGPlWg=;\n\tb=CuirEYdL2vVtJafEPuqbE/cABKNJM5fMYTqzyKsyuMPMi1a/3q037nYcYW/XoxaqNu\n\tRy9jBzVOtHJla4lpI5zHC0P8Bv/Jud9+7MwOQw0AfIgXHTfHsxxn6eFbtvGSIUhX73YC\n\tUSMa0+dJtKxgftXULXVBjXMqi0jymbL+XAbKs=", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20161025;\n\th=x-gm-message-state:mime-version:from:to:cc:subject:date:message-id\n\t:in-reply-to:references;\n\tbh=mCj3a3yGUZ0h5flR7J3TqswOq1XpSH8CvgOqnlGPlWg=;\n\tb=FNrcIdUVJVGXJEavPqvWbwVOwvKWHW/aab4BcPnUBT0l7OTDCiHrmd3/eg235S8ogN\n\t1dz+v15vgZkJ8Y9kC7KgVL3DiTz4r/hEZE1mw/qX6OCjkZ8od6qlBAlNc5e3MYnUnnyj\n\tatC6MckfGuwGsYVxe9z4Oxa0Lbc0h/P2SDjZ6S+3ksaY9Grg3j4286PKBPqf6qLW+oPn\n\tFOjrE0RTxEWjxIu+aAxkt8yLq6nZGFkciV+5BMXKOuME7W68pewTTz/k77pKZ5dIDMwC\n\tASrTZqHe5WjvFI8nQgCMRtJia2fWXPXvlPMO23uemSIlnDRQ13o30KFYmTlgxGGj1jM+\n\tk18A==", "X-Gm-Message-State": "AHPjjUiF0VrozoLefCnbBdbL26Xx7wJ5ieCAzmR7w740YXVuSGwtvcrv\n\tJ8vFxKZ0yafKLZhDST+8/vL9XN7i/dLK8xuKEylwriY/7pDvM00HH4vlL7P+Mm8ij9OQMAc2eF1\n\tirZpW", "MIME-Version": "1.0", "X-Google-Smtp-Source": "ADKCNb7zAdzKZ+clUNgO2yQW+GGInr3bEJ63kpeAxCLa838Ynu0/zaEy5hEldc+bMCYyjVu1qolPbw==", "X-Received": "by 10.99.116.19 with SMTP id p19mr3824934pgc.303.1504892304336; \n\tFri, 08 Sep 2017 10:38:24 -0700 (PDT)", "From": "Brandon Carpenter <brandon.carpenter@cypherpath.com>", "To": "qemu-devel@nongnu.org", "Date": "Fri, 8 Sep 2017 10:37:55 -0700", "Message-Id": "<20170908173801.15205-1-brandon.carpenter@cypherpath.com>", "X-Mailer": "git-send-email 2.14.1", "In-Reply-To": "<20170724184217.21381-1-brandon.carpenter@cypherpath.com>", "References": "<20170724184217.21381-1-brandon.carpenter@cypherpath.com>\n\t<20170724181544.20292-1-brandon.carpenter@cypherpath.com>", "Content-Type": "text/plain; charset=\"US-ASCII\"", "X-detected-operating-system": "by eggs.gnu.org: Genre and OS details not\n\trecognized.", "X-Received-From": "2607:f8b0:400e:c00::236", "Subject": "[Qemu-devel] [PATCH v2 0/6] Update websocket code to more fully\n\tsupport the RFC", "X-BeenThere": "qemu-devel@nongnu.org", "X-Mailman-Version": "2.1.21", "Precedence": "list", "List-Id": "<qemu-devel.nongnu.org>", "List-Unsubscribe": "<https://lists.nongnu.org/mailman/options/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>", "List-Archive": "<http://lists.nongnu.org/archive/html/qemu-devel/>", "List-Post": "<mailto:qemu-devel@nongnu.org>", "List-Help": "<mailto:qemu-devel-request@nongnu.org?subject=help>", "List-Subscribe": "<https://lists.nongnu.org/mailman/listinfo/qemu-devel>,\n\t<mailto:qemu-devel-request@nongnu.org?subject=subscribe>", "Cc": "brandon.carpenter@cypherpath.com", "Errors-To": "qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org", "Sender": "\"Qemu-devel\"\n\t<qemu-devel-bounces+incoming=patchwork.ozlabs.org@nongnu.org>" }, "content": "We've been experiencing issues where the qemu websocket server closes\nconnections from noVNC clients for no apparent reason. Debugging shows\nthat certain web browsers are injecting ping and pong frames when the\nconnection becomes idle. Some browsers send those frames without a\npayload, which also is causing closure. This patch series addresses\nthese issues by making the websocket server more conformant to RFC\n6455 - The WebSocket Protocol.\n\nRemembering the opcode is sufficient for handling fragmented frames from\nthe client, which may be introduced by an intermediary server/proxy.\nRespond to pings and ignore pongs rather than close the connection as\nmany browsers use ping/pong to test an idle connection. Close\nconnections according to the RFC, including providing a reason code and\nmessage to aid debugging of unexpected disconnects. Empty payloads\nshould not cause a disconnect.\n\nWhile updating the websocket code, several other bugs were discovered\nfor which patches are also included early in the set.\n\nBrandon Carpenter (6):\n io: Always remove an old channel watch before adding a new one\n io: Small updates in preparation for websocket changes\n io: Add support for fragmented websocket binary frames\n io: Allow empty websocket payload\n io: Ignore websocket PING and PONG frames\n io: Reply to ping frames\n\n include/io/channel-websock.h | 1 +\n io/channel-websock.c | 207 ++++++++++++++++++++++---------------------\n ui/vnc-auth-vencrypt.c | 3 +\n ui/vnc-ws.c | 6 ++\n ui/vnc.c | 4 +\n 5 files changed, 122 insertions(+), 99 deletions(-)" }