]> source.dussan.org Git - rspamd.git/commitdiff
[Minor] lua_Scanners - KAS SE fix response, av caching 3068/head
authorCarsten Rosenberg <c.rosenberg@heinlein-support.de>
Mon, 7 Oct 2019 15:08:39 +0000 (17:08 +0200)
committerCarsten Rosenberg <c.rosenberg@heinlein-support.de>
Mon, 7 Oct 2019 15:08:39 +0000 (17:08 +0200)
lualib/lua_scanners/kaspersky_se.lua

index 098e33cdf4d91373165ac146a2c7c25d61dcdc7c..a1c8992c1ece955160e6672f9570af7d9eb80478 100644 (file)
@@ -130,7 +130,7 @@ local function kaspersky_se_check(task, content, digest, rule)
       local message_fd = rspamd_util.create_file(fname)
 
       if not message_fd then
-        rspamd_logger.errx('cannot store file for savapi scan: %s', fname)
+        rspamd_logger.errx('cannot store file for kaspersky_se scan: %s', fname)
         return
       end
 
@@ -203,13 +203,12 @@ local function kaspersky_se_check(task, content, digest, rule)
           task:insert_result(rule.symbol_fail, 1.0, 'Bad HTTP code: ' .. code)
           return
         end
-        lua_util.debugm(rule.name, task, 'got reply: %s', body)
-        local data = tostring(body)
+        local data = string.gsub(tostring(body), '[\r\n%s]$', '')
         local cached
-        lua_util.debugm(rule.name, task, '%s: got reply: %s',
+        lua_util.debugm(rule.name, task, '%s: got reply data: "%s"',
             rule.log_prefix, data)
         if data == 'CLEAN' then
-          cached = 'CLEAN'
+          cached = 'OK'
           if rule['log_clean'] then
             rspamd_logger.infox(task, '%s: message or mime_part is clean',
                 rule.log_prefix)
@@ -239,9 +238,10 @@ local function kaspersky_se_check(task, content, digest, rule)
           rspamd_logger.errx(task, '%s: unhandled response: %s', rule.log_prefix, data)
           common.yield_result(task, rule, 'unhandled response:' .. data, 0.0, 'fail')
         end
-          if cached then
+
+        if cached then
           common.save_cache(task, digest, rule, cached)
-          end
+        end
 
       end
     end