# HG changeset patch
# User Kim Alvefur <zash@zash.se>
# Date 1669504238 -3600
# Node ID dec4b2e31d1c92fcd10b6c8d0d250d59521c5a15
# Parent  e384b91d0aa712c4379e7c83d3fba24130a0fcbe
mod_rest: Do not allow replies to <iq type=result> from webhooks

Replying to a result stanza could cause loops, just as replying to an
error.

No stanza.name check here but it should be fine since no other kinds of
stanzas have type=result, and the boolean logic would most definitely be
wrong on the first attempt.

diff -r e384b91d0aa7 -r dec4b2e31d1c mod_rest/mod_rest.lua
--- a/mod_rest/mod_rest.lua	Tue Nov 22 16:59:52 2022 +0100
+++ b/mod_rest/mod_rest.lua	Sun Nov 27 00:10:38 2022 +0100
@@ -500,7 +500,7 @@
 
 	local function handle_stanza(event)
 		local stanza, origin = event.stanza, event.origin;
-		local reply_allowed = stanza.attr.type ~= "error";
+		local reply_allowed = stanza.attr.type ~= "error" and stanza.attr.type ~= "result";
 		local reply_needed = reply_allowed and stanza.name == "iq";
 		local receipt;