Opened 12 years ago

Closed 12 years ago

#4976 closed defect (fixed)

[cometd] Callback transport doesn't handle advice or disconnect

Reported by: brendonh Owned by: Tom Trenka
Priority: high Milestone: 1.0
Component: Dojox Version: 0.9
Keywords: cometd Cc:
Blocked By: Blocking:

Description

The longpoll transport has support for the important advice fields ("reconnect" and "interval"), but the callback transport ignores them.

As a result, if you disconnect while using the callback transport, the client goes into an endless loop trying to connect again, because tunnelCollapse doesn't notice.

Attachments (3)

cometd.patch​ (5.7 KB) - added by gregwilkins 12 years ago.
fix plus disconnect
cometd.2.patch​ (8.8 KB) - added by gregwilkins 12 years ago.
3rd version of the patch - now with lots of comments cleaned up and reconnect=handshake fixed
cometd.3.patch​ (9.3 KB) - added by gregwilkins 12 years ago.
patch 4 with disconnect fix

Download all attachments as: .zip

Change History (4)

Changed 12 years ago by gregwilkins

Attachment: cometd.patch​ added

fix plus disconnect

Changed 12 years ago by gregwilkins

Attachment: cometd.2.patch​ added

3rd version of the patch - now with lots of comments cleaned up and reconnect=handshake fixed

Changed 12 years ago by gregwilkins

Attachment: cometd.3.patch​ added

patch 4 with disconnect fix

comment:1 Changed 12 years ago by gregwilkins

Resolution: → fixed
Status: new → closed
Note: See TracTickets for help on using tickets.