From d473407ec97107022ccbdd366559001b1ec162d8 Mon Sep 17 00:00:00 2001 From: Alexandre Flament Date: Sun, 17 Jan 2021 16:14:16 +0100 Subject: [fix] checker: fix engine statistics Without this commit, the URL /stats/errors shows percentage above 100% after the checker has run. --- searx/search/checker/impl.py | 3 +++ searx/search/processors/abstract.py | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) (limited to 'searx/search') diff --git a/searx/search/checker/impl.py b/searx/search/checker/impl.py index 71a941f73..244536f1b 100644 --- a/searx/search/checker/impl.py +++ b/searx/search/checker/impl.py @@ -4,6 +4,7 @@ import typing import types import functools import itertools +import threading from time import time from urllib.parse import urlparse @@ -377,6 +378,8 @@ class Checker: engineref_category = search_query.engineref_list[0].category params = self.processor.get_params(search_query, engineref_category) if params is not None: + with threading.RLock(): + self.processor.engine.stats['sent_search_count'] += 1 self.processor.search(search_query.query, params, result_container, time(), 5) return result_container diff --git a/searx/search/processors/abstract.py b/searx/search/processors/abstract.py index eb8d296ec..3a853d49e 100644 --- a/searx/search/processors/abstract.py +++ b/searx/search/processors/abstract.py @@ -1,13 +1,13 @@ # SPDX-License-Identifier: AGPL-3.0-or-later -from abc import abstractmethod +from abc import abstractmethod, ABC from searx import logger logger = logger.getChild('searx.search.processor') -class EngineProcessor: +class EngineProcessor(ABC): def __init__(self, engine, engine_name): self.engine = engine -- cgit v1.2.3 From 67a1aab0d5b8ab93b60be5da2390039f6c861505 Mon Sep 17 00:00:00 2001 From: Alexandre Flament Date: Sun, 17 Jan 2021 16:56:36 +0100 Subject: [fix] /stats/checker : remove the timestamp field when the checker is disabled --- searx/search/checker/background.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'searx/search') diff --git a/searx/search/checker/background.py b/searx/search/checker/background.py index e41bff5f5..22a51b8bb 100644 --- a/searx/search/checker/background.py +++ b/searx/search/checker/background.py @@ -98,7 +98,7 @@ def initialize(): signal.signal(signal.SIGUSR1, _signal_handler) # disabled by default - _set_result({'status': 'disabled'}) + _set_result({'status': 'disabled'}, include_timestamp=False) # special case when debug is activate if searx_debug and settings.get('checker', {}).get('off_when_debug', True): -- cgit v1.2.3