Opened 12 years ago

Closed 12 years ago

#4880 closed defect (fixed)

[cometd][patch][ccla] Increment "id" field on client side, use it in all messages.

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

Description

This patch adds a single "messageId" attribute to the main cometd object and has all transports use (and increment) it (as "id") when sending messages. I had to change a couple of for loops to make the ordering come out right.

Servers SHOULD return this ID in replies but I don't think implementations have caught up yet. When they do, cometd.js can start using it in place of stuff like pendingSubscriptions.

Attachments (1)

patch.txt (3.5 KB) - added by brendonh 12 years ago.

Download all attachments as: .zip

Change History (2)

Changed 12 years ago by brendonh

Attachment: patch.txt added

comment:1 Changed 12 years ago by alex

Resolution: fixed
Status: newclosed

(In [11153]) merging another good patch from Brendan Hogger (CLA on file). Adds ID support to cometd.js messages. Fixes #4880

Note: See TracTickets for help on using tickets.