Jetty

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

Ascending order - Click to sort in descending order
Hide
Permalink
Greg Wilkins added a comment -

Fixed.

thanks

Show
Greg Wilkins added a comment - Fixed. thanks
Hide
Permalink
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
Permalink
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
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: