Details

    • Type: Bug
    • Status: Done
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 0.9.9
    • Component/s: guacamole
    • Labels:
      None

      Description

      If the WebSocket tunnel is used, it cannot possibly be aware of any connection-related errors, including lack of authorization, as connection errors are passed back through the HTTP request. The WebSocket API does not allow reading of HTTP headers, and the WebSocket spec prohibits exposure of such data.

      Probably need to pass errors back in a WebSocket-specific way.

        Attachments

          Issue Links

            Expenses

              Activity

              Hide
              mike.jumper Michael Jumper added a comment -

              Mitigated by the fact that current use of the WebSocket tunnel is chained to the HTTP tunnel, so any connection failure with WebSocket is immediately followed by a retry with the HTTP tunnel, which fails identically, resulting in the error passing through.

              Show
              mike.jumper Michael Jumper added a comment - Mitigated by the fact that current use of the WebSocket tunnel is chained to the HTTP tunnel, so any connection failure with WebSocket is immediately followed by a retry with the HTTP tunnel, which fails identically, resulting in the error passing through.
              Hide
              mike.jumper Michael Jumper added a comment -

              This is fixed by GUAC-1427. The actual Guacamole connection is not made until the WebSocket connection is fully established.

              Show
              mike.jumper Michael Jumper added a comment - This is fixed by GUAC-1427 . The actual Guacamole connection is not made until the WebSocket connection is fully established.

                People

                • Assignee:
                  Unassigned
                  Reporter:
                  mike.jumper Michael Jumper
                • Votes:
                  0 Vote for this issue
                  Watchers:
                  1 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved: