Opened 13 years ago

Closed 13 years ago

#27 closed defect (fixed)

Patch to fix some issues with pubsub protocol compliance

Reported by: caedes Owned by: ralphm
Priority: normal Milestone: 0.5
Component: generic Keywords:
Cc: caedes, ralphm

Description

The patch handles two issues:

  • Made the create command include a configure node, required by the protocol.
  • Made delete command belong to owner pubsub namespace.

Attachments (1)

pubsub.patch (2.0 KB) - added by caedes 13 years ago.
File fixing the pubsub errors, and the unit tests.

Download all attachments as: .zip

Change History (4)

Changed 13 years ago by caedes

File fixing the pubsub errors, and the unit tests.

comment:1 Changed 13 years ago by ralphm

  • Cc caedes ralphm added
  • Milestone set to 0.5
  • Status changed from new to assigned

Actually, the topic of an empty <configure/> element on node creation is being discussed on the pubsub mailinglist. At the very least it is a bug in the current version of the specification that this element is required for creating nodes.

As for the namespace of the delete element, yeah, that is indeed a bug in wokkel. Thanks for the patch.

comment:2 Changed 13 years ago by ralphm

(In [74]) Partially apply supplied patch by caedes, re #27.

comment:3 Changed 13 years ago by ralphm

  • Resolution set to fixed
  • Status changed from assigned to closed

(In [75]) Use correct namespace for delete node requests.

Author: caedes. Reviewer: ralphm. Fixes #27.

Note: See TracTickets for help on using tickets.