Difference between revisions of "PubSubIssues"

Jump to navigation Jump to search
1,947 bytes added ,  09:32, 5 April 2015
 
 
(6 intermediate revisions by 2 users 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 24: Line 30:
* 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.
* 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 is not specified (Section 5.3 and 5.5) ===
=== 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.
 
=== Redundant features for Item Retraction ===
 
The latest revision introduced a feature 'delete-items', while 'retract-items' was already there.  Ralphm proposed to the list:
 
<blockquote>
[...] Reading the text around it, I feel it is
confusing, too. I'd rather go with talking about 'retracting items' and
'deleting nodes' thoughout, and not talk about 'deleting items'.
</blockquote>
 
<blockquote>
I think that many (all?) clients ignore this feature for discovery
altogether, so what about making 'delete-items' a thing that servers
SHOULD also advertise along with 'retract-items', but have clients
depend on 'retract-items' exclusively?
</blockquote>
 
=== No text on #publish_model ===
 
A few examples mention an option #publish_model, which seems useful, but there's no actual text describing behaviour of said option.
 
=== Entity node creation authorization request (Section 8.1) ===


It is not specified what the PubSub service should return if the client performs a disco info or disco items request on a non-existent PubSub 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).
There's no way to know if an entity can create (or not) a node before actually create it.
This issue bring a bad UI experience on some Pubsub clients (we display an error after the form creation submission).


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

edits

Navigation menu