Jetty
  1. Jetty
  2. JETTY-368

Disconnect keeps polling connection open

    Details

    • Type: Bug Bug
    • Status: Resolved Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 6.1.4rc1
    • Component/s: Bayeux
    • Labels:
      None
    • Number of attachments :
      0

      Description

      A disconnect is send over a second connection, that means the long polling (RECONNECT) connection must be closed independently. Currently that's not the case, the connection is open until the reconnect timeout happens.
      I added a continuation.resume to the remove method of the continuationclient and everything seems to work. however I do not have the confidence to say that this might not have any side impacts. greg, can you have a look at it?

        Activity

        Hide
        Greg Wilkins added a comment -

        Fixed.

        thanks

        Show
        Greg Wilkins added a comment - Fixed. thanks
        Hide
        Markus Joschko added a comment -

        you put a resume in the remove method of clientimpl. The one i add is in continuationClients remove method. Was the duplication done by purpose?

        Show
        Markus Joschko added a comment - you put a resume in the remove method of clientimpl. The one i add is in continuationClients remove method. Was the duplication done by purpose?
        Hide
        Greg Wilkins added a comment -

        I meant to remove yours, but forgot.
        I think the resume call is generic to any implementation that implements the resume.

        cheers

        Show
        Greg Wilkins added a comment - I meant to remove yours, but forgot. I think the resume call is generic to any implementation that implements the resume. cheers

          People

          • Assignee:
            Unassigned
            Reporter:
            Markus Joschko
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: