25
1
mirror of https://github.com/processone/ejabberd.git synced 2024-12-26 17:38:45 +01:00

Merge pull request #2526 from nosnilmot/enforce-pubsub-option-attributes

Enforce pubsub option required/rejected attributes
This commit is contained in:
Evgeny Khramtsov 2018-07-14 16:42:15 +03:00 committed by GitHub
commit 711c5c0d54
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 17 additions and 2 deletions

View File

@ -25,7 +25,7 @@
{fast_tls, ".*", {git, "https://github.com/processone/fast_tls", {tag, "1.0.23"}}},
{stringprep, ".*", {git, "https://github.com/processone/stringprep", {tag, "1.0.12"}}},
{fast_xml, ".*", {git, "https://github.com/processone/fast_xml", {tag, "1.1.32"}}},
{xmpp, ".*", {git, "https://github.com/processone/xmpp", "c98ee04"}},
{xmpp, ".*", {git, "https://github.com/processone/xmpp", "007a716"}},
{fast_yaml, ".*", {git, "https://github.com/processone/fast_yaml", {tag, "1.0.15"}}},
{jiffy, ".*", {git, "https://github.com/davisp/jiffy", {tag, "0.14.8"}}},
{p1_oauth2, ".*", {git, "https://github.com/processone/p1_oauth2", {tag, "0.6.3"}}},

View File

@ -1138,8 +1138,17 @@ iq_pubsub(Host, Access, #iq{from = From, type = IQType, lang = Lang,
{set, #pubsub{subscribe = #ps_subscribe{node = Node, jid = JID},
options = Options, _ = undefined}} ->
Config = case Options of
#ps_options{xdata = XData} ->
#ps_options{xdata = XData, jid = undefined, node = <<>>} ->
decode_subscribe_options(XData, Lang);
#ps_options{xdata = _XData, jid = _JID, node = <<>>} ->
Txt = <<"jid not allowed here">>,
{error, xmpp:err_bad_request(Txt, Lang)};
#ps_options{xdata = _XData, jid = undefined, node = _NodeID} ->
Txt = <<"node not allowed here">>,
{error, xmpp:err_bad_request(Txt, Lang)};
#ps_options{xdata = _XData, jid = _JID, node = _NodeID} ->
Txt = <<"jid and node not allowed here">>,
{error, xmpp:err_bad_request(Txt, Lang)};
_ ->
[]
end,
@ -1165,6 +1174,12 @@ iq_pubsub(Host, Access, #iq{from = From, type = IQType, lang = Lang,
{get, #pubsub{affiliations = {Node, _}, _ = undefined}} ->
Plugins = config(serverhost(Host), plugins),
get_affiliations(Host, Node, From, Plugins);
{_, #pubsub{options = #ps_options{jid = undefined}, _ = undefined}} ->
Txt = <<"jid required">>,
{error, extended_error(xmpp:err_bad_request(Txt, Lang), err_jid_required())};
{_, #pubsub{options = #ps_options{node = <<>>}, _ = undefined}} ->
Txt = <<"nodeid required">>,
{error, extended_error(xmpp:err_bad_request(Txt, Lang), err_nodeid_required())};
{get, #pubsub{options = #ps_options{node = Node, subid = SubId, jid = JID},
_ = undefined}} ->
get_options(Host, Node, JID, SubId, Lang);