From a8ce138dd2367f39e3a90857869245a197bb9f8c Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jonas=20Sch=C3=A4fer?= Date: Tue, 20 Aug 2019 20:11:10 +0200 Subject: XEP-0410: set not-in-the-room error condition to not-acceptable MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit XEP-0410 demands that. This changes the error condition for all pings to the room, even those which aren’t self-pings, if the sender is not joined. The conditions of XEP-0410 were based on observing existing implementations. --- src/bridge/bridge.cpp | 2 +- tests/end_to_end/__main__.py | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/bridge/bridge.cpp b/src/bridge/bridge.cpp index f065930..4d4e9de 100644 --- a/src/bridge/bridge.cpp +++ b/src/bridge/bridge.cpp @@ -743,7 +743,7 @@ void Bridge::send_irc_participant_ping_request(const Iid& iid, const std::string IrcChannel* chan = irc->get_channel(iid.get_local()); if (!chan->joined || !this->is_resource_in_chan(iid.to_tuple(), from.resource)) { - this->xmpp.send_stanza_error("iq", to_jid, from_jid, iq_id, "cancel", "not-allowed", + this->xmpp.send_stanza_error("iq", to_jid, from_jid, iq_id, "cancel", "not-acceptable", "", true); return; } diff --git a/tests/end_to_end/__main__.py b/tests/end_to_end/__main__.py index 67dffc9..f6cf38b 100644 --- a/tests/end_to_end/__main__.py +++ b/tests/end_to_end/__main__.py @@ -1543,14 +1543,14 @@ if __name__ == '__main__': ""), # Immediately receive an error partial(expect_stanza, - "/iq[@from='#nil%{irc_server_one}/{nick_one}'][@type='error'][@to='{jid_one}/{resource_one}'][@id='first_ping']/error/stanza:not-allowed"), + "/iq[@from='#nil%{irc_server_one}/{nick_one}'][@type='error'][@to='{jid_one}/{resource_one}'][@id='first_ping']/error/stanza:not-acceptable"), # Send a ping to ourself, in a muc where we are, but not this resource partial(send_stanza, ""), # Immediately receive an error partial(expect_stanza, - "/iq[@from='#foo%{irc_server_one}/{nick_one}'][@type='error'][@to='{jid_one}/{resource_two}'][@id='first_ping']/error/stanza:not-allowed"), + "/iq[@from='#foo%{irc_server_one}/{nick_one}'][@type='error'][@to='{jid_one}/{resource_two}'][@id='first_ping']/error/stanza:not-acceptable"), ]), Scenario("self_ping_on_real_channel", [ -- cgit v1.2.3