Difference between revisions of "PubSubIssues"

From XMPP WIKI
Jump to navigation Jump to search
(One intermediate revision by the same user not shown)
Line 4: Line 4:


Note: these issues are keyed to the section numbers in version 1.13 of XEP-0060.
Note: these issues are keyed to the section numbers in version 1.13 of XEP-0060.
=== Temporary Subscriptions is not discoverable (Section 12.4) ===
http://xmpp.org/extensions/xep-0060.html#impl-tempsub describes temprary subscriptions for a pub/sub node. It is not mentioned anywhere in the XEP-0060 how to discover this feature (pubsub#tempsub).


=== Affiliation Change Notifications (Section 8.9.4) ===
=== Affiliation Change Notifications (Section 8.9.4) ===
Line 27: Line 33:


Would be good if the PubSub spec would have a reference to the service discovery error conditions if a disco request is send to a non existent node. I've reports from users that some implementations return a non-error result, which e.g. results in Smack returning a NotAPubSubNodeException instead of a XMPPErrorException (which is what most would expect). So it's not clear for some implementors what do in this case (which is likely because the relevant information is found in a different XEP).
Would be good if the PubSub spec would have a reference to the service discovery error conditions if a disco request is send to a non existent node. I've reports from users that some implementations return a non-error result, which e.g. results in Smack returning a NotAPubSubNodeException instead of a XMPPErrorException (which is what most would expect). So it's not clear for some implementors what do in this case (which is likely because the relevant information is found in a different XEP).
=== Optional redirect URI for <delete/> not discoverable (Section 8.4) ===
Server support for <redirect/> is optional for servers, yet clients have no way of discovering if the server supports it.  Also, it doesn't state the maximum number of <redirect/> elements allowed, although it seems to suggest it's one.  Proposed solution:
* Add feature delete-nodes-redirect (OPTIONAL)
* Modify language to state there can be only one redirect element.


== XEP-0163: PEP ==
== XEP-0163: PEP ==

Revision as of 09:50, 8 January 2015

This page lists known issues with the Publish-Subscribe specifications (XEP-0060, XEP-0163, XEP-248).

XEP-0060: PubSub

Note: these issues are keyed to the section numbers in version 1.13 of XEP-0060.

Temporary Subscriptions is not discoverable (Section 12.4)

http://xmpp.org/extensions/xep-0060.html#impl-tempsub describes temprary subscriptions for a pub/sub node. It is not mentioned anywhere in the XEP-0060 how to discover this feature (pubsub#tempsub).


Affiliation Change Notifications (Section 8.9.4)

Details and Suggested Changes

Currently, the notification for affiliation changes uses the affiliation element found in the pubsub namespace. This conflicts with all other notifications, that refer to matching elements defined in the pubsub#event namespace. Affiliations are genetically similar to subscriptions and should mimic how subscription notifications are handled. Resolving this requires the following changes:

  • Update the schema, in section 17.4, to include the affiliation element as an option under the event element.
  • Update the schema, in section 17.4, to include a definition for the affiliation element.
    • The new affiliation element should include a node, jid and affiliation attribute.
  • Update Example 208 to more closely match Example 194.
    • Replace pubsub element with the event element.
    • Remove the affiliations element
    • Update the affiliation element to include the node attribute that was in the affiliations element
  • Investigate whether the body element can be used to provide change related text to any notification messages. If it can, each notification area should include this information or the information should be place in a general location.
  • Section 7.1.2.3 says: "If configured to do so, the service can include the publisher of the item when it generates event notifications." but there is no way to configure this bevahiour described. Probably need to add appropriate discovery feature + configuration form item.

Expected response to disco info and disco items request to a non existing node (Section 5.3 and 5.5)

Would be good if the PubSub spec would have a reference to the service discovery error conditions if a disco request is send to a non existent node. I've reports from users that some implementations return a non-error result, which e.g. results in Smack returning a NotAPubSubNodeException instead of a XMPPErrorException (which is what most would expect). So it's not clear for some implementors what do in this case (which is likely because the relevant information is found in a different XEP).

Optional redirect URI for <delete/> not discoverable (Section 8.4)

Server support for <redirect/> is optional for servers, yet clients have no way of discovering if the server supports it. Also, it doesn't state the maximum number of <redirect/> elements allowed, although it seems to suggest it's one. Proposed solution:

  • Add feature delete-nodes-redirect (OPTIONAL)
  • Modify language to state there can be only one redirect element.

XEP-0163: PEP

No known issues.

XEP-0248: Collections

Needs a general overhaul.