Update session management logic in lua plugins

The previous TODO comment has been addressed and removed from the monitoring plugin. In the callback plugin, logic has been modified that handles session expiry and cleanup. Now, if an IMAP command is "NOOP", the session will expire in 24 hours.

Signed-off-by: Christian Roessner <c@roessner.co>

server/lua-plugins.d/callback/callback.lua

@@ -48,6 +48,9 @@ function nauthilus_run_callback()
         result.state = "client disconnected"
         result.dovecot_session = "unknown"
 
+        ---@type boolean is_cmd_noop
+        local is_cmd_noop = false
+
         ---@param k table
         ---@param v any
         for k, v in pairs(body_table) do
@@ -81,6 +84,10 @@ function nauthilus_run_callback()
                             result.remote_ip = field_value
                         elseif field_name == "remote_port" then
                             result.remote_port = field_value
+                        elseif field_name == "cmd_name" then
+                            if field_value == "NOOP" then
+                                is_cmd_noop = true
+                            end
                         end
                     end
                 end
@@ -89,14 +96,21 @@ function nauthilus_run_callback()
 
         if result.category == "service:imap" or result.category == "service:lmtp" then
             if result.dovecot_session ~= "unknown" then
-                -- Cleanup dovecot session
-                ---@type string deleted
-                ---@type string err_redis_hdel
-                local deleted, err_redis_hdel = nauthilus.redis_hdel("ntc:DS:" .. crypto.md5(result.user), result.dovecot_session)
-                if err_redis_hdel ~= nil then
-                    result.removed_session_failure = err_redis_hdel
+                ---@type string redis_key
+                local redis_key = "ntc:DS:" .. crypto.md5(result.user)
+
+                if is_cmd_noop then
+                    nauthilus.redis_expire(redis_key, 86400)
                 else
-                    result.removed_session = deleted
+                    -- Cleanup dovecot session
+                    ---@type string deleted
+                    ---@type string err_redis_hdel
+                    local deleted, err_redis_hdel = nauthilus.redis_hdel(redis_key, result.dovecot_session)
+                    if err_redis_hdel ~= nil then
+                        result.removed_session_failure = err_redis_hdel
+                    else
+                        result.removed_session = deleted
+                    end
                 end
             end
 

server/lua-plugins.d/filters/monitoring.lua

@@ -2,7 +2,7 @@ local crypto = require("crypto")
 
 function nauthilus_call_filter(request)
     ---@return string
-    local function get_dovecot_session() -- TODO: Trach server port as well
+    local function get_dovecot_session()
         ---@type table headers
         local headers = nauthilus:get_all_http_request_headers()
         ---@param header_name string