tyrus
  1. tyrus
  2. TYRUS-222

HandshakeException when checking "Connection" header from Firefox client

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.2
    • Fix Version/s: 1.2.1
    • Component/s: None
    • Labels:
      None
    • Environment:

      FireFox 17.0.1

      Description

      It seems the org.glassfish.tyrus.websockets.HandShake class validates that the "Connection" header is equal to "Upgrade", but the actual header is "keep-alive, upgrade".

        Activity

        Hide
        jmarine added a comment -

        I have reopened issue 55 because in versions 1.1 and 1.2, the org.glassfish.tyrus.websockets.HandShake only checks for the first value of the "Connection" header ("keep-alive" value, instead of all the complete header value: "keep-alive, upgrade")

        Line 97:
        checkForHeader(request.getFirstHeaderValue(WebSocketEngine.CONNECTION), WebSocketEngine.CONNECTION, WebSocketEngine.UPGRADE);

        Show
        jmarine added a comment - I have reopened issue 55 because in versions 1.1 and 1.2, the org.glassfish.tyrus.websockets.HandShake only checks for the first value of the "Connection" header ("keep-alive" value, instead of all the complete header value: "keep-alive, upgrade") Line 97: checkForHeader(request.getFirstHeaderValue(WebSocketEngine.CONNECTION), WebSocketEngine.CONNECTION, WebSocketEngine.UPGRADE);

          People

          • Assignee:
            Pavel Bucek
            Reporter:
            jmarine
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: