Mercurial > prosody-hg
comparison plugins/mod_pubsub/pubsub.lib.lua @ 9120:01fc8ba0a7fa
mod_pubsub: Use util.pubsub API for retrieving node config instead
Normalizes access control to the 'get_configuration' capability
Thanks pep.
| author | Kim Alvefur <zash@zash.se> |
|---|---|
| date | Sun, 05 Aug 2018 19:12:49 +0200 |
| parents | 9f36f5c14545 |
| children | a69dc9c729ec |
comparison
equal
deleted
inserted
replaced
| 9119:5444d4a8ad90 | 9120:01fc8ba0a7fa |
|---|---|
| 630 if not node then | 630 if not node then |
| 631 origin.send(pubsub_error_reply(stanza, "nodeid-required")); | 631 origin.send(pubsub_error_reply(stanza, "nodeid-required")); |
| 632 return true; | 632 return true; |
| 633 end | 633 end |
| 634 | 634 |
| 635 if not service:may(node, stanza.attr.from, "configure") then | 635 local ok, node_config = service:get_node_config(node, stanza.attr.from); |
| 636 origin.send(pubsub_error_reply(stanza, "forbidden")); | 636 if not ok then |
| 637 return true; | 637 origin.send(pubsub_error_reply(stanza, node_config)); |
| 638 end | 638 return true; |
| 639 | 639 end |
| 640 local node_obj = service.nodes[node]; | 640 |
| 641 if not node_obj then | |
| 642 origin.send(pubsub_error_reply(stanza, "item-not-found")); | |
| 643 return true; | |
| 644 end | |
| 645 | |
| 646 local node_config = node_obj.config; | |
| 647 local pubsub_form_data = config_to_xep0060(node_config); | 641 local pubsub_form_data = config_to_xep0060(node_config); |
| 648 local reply = st.reply(stanza) | 642 local reply = st.reply(stanza) |
| 649 :tag("pubsub", { xmlns = xmlns_pubsub_owner }) | 643 :tag("pubsub", { xmlns = xmlns_pubsub_owner }) |
| 650 :tag("configure", { node = node }) | 644 :tag("configure", { node = node }) |
| 651 :add_child(node_config_form:form(pubsub_form_data)); | 645 :add_child(node_config_form:form(pubsub_form_data)); |
