From 8a949f83e262f5ba4010c9ddb5cb7640c3c46c86 Mon Sep 17 00:00:00 2001 From: Kim Alvefur Date: Sat, 20 Nov 2021 23:54:43 +0100 Subject: mod_csi_simple: Skip initiating flush in all but inactive state Both in the flushing and active states the right thing to do is skip directly to returning the data. Also in any unknown state, like if the filter is somehow left behind on module unload. --- plugins/mod_csi_simple.lua | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'plugins/mod_csi_simple.lua') diff --git a/plugins/mod_csi_simple.lua b/plugins/mod_csi_simple.lua index a2aeceef..2420705a 100644 --- a/plugins/mod_csi_simple.lua +++ b/plugins/mod_csi_simple.lua @@ -143,7 +143,7 @@ end local function flush_buffer(data, session) local ctr = session.csi_counter or 0; - if ctr == 0 or session.state == "flushing" then return data end + if ctr == 0 or session.state ~= "inactive" then return data end session.log("debug", "Flushing buffer (%s; queue size is %d)", "client activity", session.csi_counter); session.state = "flushing"; module:fire_event("csi-flushing", { session = session }); -- cgit v1.2.3