mirror of
https://github.com/searxng/searxng
synced 2024-01-01 19:24:07 +01:00
Merge 27aa9c4cb1 into 272e39893d
This commit is contained in:
commit
982100e3a7
4 changed files with 54 additions and 19 deletions
|
|
@ -3,6 +3,8 @@
|
|||
|
||||
"""
|
||||
|
||||
import logging
|
||||
from logging import Logger
|
||||
import threading
|
||||
from abc import abstractmethod, ABC
|
||||
from timeit import default_timer
|
||||
|
|
@ -58,12 +60,13 @@ class SuspendedStatus:
|
|||
class EngineProcessor(ABC):
|
||||
"""Base classes used for all types of request processors."""
|
||||
|
||||
__slots__ = 'engine', 'engine_name', 'lock', 'suspended_status', 'logger'
|
||||
__slots__ = 'engine', 'engine_name', 'lock', 'suspended_status', 'logger', 'log_engine_exc_info'
|
||||
|
||||
def __init__(self, engine, engine_name: str):
|
||||
self.engine = engine
|
||||
self.engine_name = engine_name
|
||||
self.logger = engines[engine_name].logger
|
||||
self.logger: Logger = engines[engine_name].logger
|
||||
self.log_engine_exc_info = True
|
||||
key = get_network(self.engine_name)
|
||||
key = id(key) if key else self.engine_name
|
||||
self.suspended_status = SUSPENDED_STATUS.setdefault(key, SuspendedStatus())
|
||||
|
|
@ -82,6 +85,10 @@ class EngineProcessor(ABC):
|
|||
def has_initialize_function(self):
|
||||
return hasattr(self.engine, 'init')
|
||||
|
||||
@property
|
||||
def metrics_log_level(self) -> int:
|
||||
return logging.WARN if self.log_engine_exc_info else logging.NOTSET
|
||||
|
||||
def handle_exception(self, result_container, exception_or_message, suspend=False):
|
||||
# update result_container
|
||||
if isinstance(exception_or_message, BaseException):
|
||||
|
|
@ -95,9 +102,9 @@ class EngineProcessor(ABC):
|
|||
# metrics
|
||||
counter_inc('engine', self.engine_name, 'search', 'count', 'error')
|
||||
if isinstance(exception_or_message, BaseException):
|
||||
count_exception(self.engine_name, exception_or_message)
|
||||
count_exception(self.engine_name, exception_or_message, log_level=self.metrics_log_level)
|
||||
else:
|
||||
count_error(self.engine_name, exception_or_message)
|
||||
count_error(self.engine_name, exception_or_message, log_level=self.metrics_log_level)
|
||||
# suspend the engine ?
|
||||
if suspend:
|
||||
suspended_time = None
|
||||
|
|
|
|||
|
|
@ -127,6 +127,7 @@ class OnlineProcessor(EngineProcessor):
|
|||
'{} redirects, maximum: {}'.format(len(response.history), soft_max_redirects),
|
||||
(status_code, reason, hostname),
|
||||
secondary=True,
|
||||
log_level=self.metrics_log_level,
|
||||
)
|
||||
|
||||
return response
|
||||
|
|
@ -177,20 +178,21 @@ class OnlineProcessor(EngineProcessor):
|
|||
self.logger.exception(
|
||||
"requests exception (search duration : {0} s, timeout: {1} s) : {2}".format(
|
||||
default_timer() - start_time, timeout_limit, e
|
||||
)
|
||||
),
|
||||
exc_info=self.log_engine_exc_info,
|
||||
)
|
||||
except SearxEngineCaptchaException as e:
|
||||
self.handle_exception(result_container, e, suspend=True)
|
||||
self.logger.exception('CAPTCHA')
|
||||
self.logger.exception('CAPTCHA', exc_info=self.log_engine_exc_info)
|
||||
except SearxEngineTooManyRequestsException as e:
|
||||
self.handle_exception(result_container, e, suspend=True)
|
||||
self.logger.exception('Too many requests')
|
||||
self.logger.exception('Too many requests', exc_info=self.log_engine_exc_info)
|
||||
except SearxEngineAccessDeniedException as e:
|
||||
self.handle_exception(result_container, e, suspend=True)
|
||||
self.logger.exception('SearXNG is blocked')
|
||||
self.logger.exception('SearXNG is blocked', exc_info=self.log_engine_exc_info)
|
||||
except Exception as e: # pylint: disable=broad-except
|
||||
self.handle_exception(result_container, e)
|
||||
self.logger.exception('exception : {0}'.format(e))
|
||||
self.logger.exception('exception : {0}'.format(e), exc_info=self.log_engine_exc_info)
|
||||
|
||||
def get_default_tests(self):
|
||||
tests = {}
|
||||
|
|
|
|||
Loading…
Add table
Add a link
Reference in a new issue