Comment 2 for bug 1715186

Revision history for this message
Daniel Berrange (berrange) wrote :

Will be included for 2.11 in

commit 3a3f8705962c8c8a47a9b981ffd5aab7274ad508
Author: Daniel P. Berrange <email address hidden>
Date: Wed Sep 6 11:38:36 2017 +0100

    io: include full error message in websocket handshake trace

    When the websocket handshake fails it is useful to log the real
    error message via the trace points for debugging purposes.

    Fixes bug: #1715186

    Reviewed-by: Philippe Mathieu-Daudé <email address hidden>
    Signed-off-by: Daniel P. Berrange <email address hidden>

commit f69a8bde29354493ff8aea64cc9cb3b531d16337
Author: Daniel P. Berrange <email address hidden>
Date: Wed Sep 6 11:33:17 2017 +0100

    io: send proper HTTP response for websocket errors

    When any error occurs while processing the websockets handshake,
    QEMU just terminates the connection abruptly. This is in violation
    of the HTTP specs and does not help the client understand what they
    did wrong. This is particularly bad when the client gives the wrong
    path, as a "404 Not Found" would be very helpful.

    Refactor the handshake code so that it always sends a response to
    the client unless there was an I/O error.

    Fixes bug: #1715186

    Reviewed-by: Philippe Mathieu-Daudé <email address hidden>
    Signed-off-by: Daniel P. Berrange <email address hidden>