diff options
author | Matthew Wild <mwild1@gmail.com> | 2018-04-29 22:15:44 +0100 |
---|---|---|
committer | Matthew Wild <mwild1@gmail.com> | 2018-04-29 22:15:44 +0100 |
commit | d62d872cd329dd0bfb487258a21a0532f24b01fa (patch) | |
tree | 7d5d7fcc5fa70133b64853b677bc90c62f04af0b | |
parent | bf5d8303ce7de0a782c3472627992b963e5ce03c (diff) | |
download | prosody-d62d872cd329dd0bfb487258a21a0532f24b01fa.tar.gz prosody-d62d872cd329dd0bfb487258a21a0532f24b01fa.zip |
mod_bosh: Fix for 7be8f649d97d to skip error handling and allow other modules to handle the request
-rw-r--r-- | plugins/mod_bosh.lua | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/plugins/mod_bosh.lua b/plugins/mod_bosh.lua index c662fa5c..68922ebd 100644 --- a/plugins/mod_bosh.lua +++ b/plugins/mod_bosh.lua @@ -191,8 +191,10 @@ function handle_POST(event) return true; -- Inform http server we shall reply later end - elseif response.finished then - return; -- A response has been sent already + elseif response.finished or context.ignore_request then + -- A response has been sent already, or we're ignoring this request + -- (e.g. so a different instance of the module can handle it) + return; end module:log("warn", "Unable to associate request with a session (incomplete request?)"); local close_reply = st.stanza("body", { xmlns = xmlns_bosh, type = "terminate", @@ -269,7 +271,8 @@ function stream_callbacks.streamopened(context, attr) elseif to_host ~= module.host then -- Could be meant for a different instance of the module -- if multiple instances are loaded with the same URL then this can happen - return; --> 404 + context.ignore_request = true; + return; end if not rid or (not wait and attr.wait or wait < 0 or wait % 1 ~= 0) then log("debug", "BOSH client sent invalid rid or wait attributes: rid=%s, wait=%s", tostring(attr.rid), tostring(attr.wait)); |