]> source.dussan.org Git - rspamd.git/commitdiff
[Minor] Fix HTTP status code 3389/head
authormoisseev <moiseev@mezonplus.ru>
Wed, 3 Jun 2020 16:41:52 +0000 (19:41 +0300)
committermoisseev <moiseev@mezonplus.ru>
Wed, 3 Jun 2020 16:41:52 +0000 (19:41 +0300)
We should treat invalid selector as a negative result of the check.

rules/controller/selectors.lua

index 58065fc85b505baecdfd0c72ec2700ab7c768fe4..905757c76c439ba616fa97295e7e842eda2283a7 100644 (file)
@@ -30,11 +30,7 @@ local function handle_check_selector(_, conn, req_params)
   if req_params.selector then
     local selector = lua_selectors.create_selector_closure(rspamd_config,
         req_params.selector, '', true)
-    if not selector then
-      conn:send_error(500, 'invalid selector')
-    else
-      conn:send_ucl({success = true})
-    end
+      conn:send_ucl({success = selector and true})
   else
     conn:send_error(404, 'missing selector')
   end
@@ -74,4 +70,4 @@ return {
     enable = true,
     need_task = true,
   }
-}
\ No newline at end of file
+}