From 060975fdfb8e6c1eed74af4a606aad0e344b204c Mon Sep 17 00:00:00 2001 From: Jordi Riera Date: Tue, 12 Feb 2019 10:20:58 -0500 Subject: [PATCH] monitorin_log_request: _dispatch is a classmethod --- monitoring_log_requests/__manifest__.py | 4 ++-- monitoring_log_requests/models/ir_http.py | 25 ++++++++++++++--------- 2 files changed, 17 insertions(+), 12 deletions(-) diff --git a/monitoring_log_requests/__manifest__.py b/monitoring_log_requests/__manifest__.py index 0f36d82..26a75b1 100644 --- a/monitoring_log_requests/__manifest__.py +++ b/monitoring_log_requests/__manifest__.py @@ -3,8 +3,8 @@ {'name': 'Monitoring: Requests Logging', - 'version': '12.0.1.0.0', - 'author': 'Camptocamp,Odoo Community Association (OCA)', + 'version': '12.0.2.0.0', + 'author': 'Camptocamp,Numigi,Odoo Community Association (OCA)', 'license': 'AGPL-3', 'category': 'category', 'depends': ['base', 'web'], diff --git a/monitoring_log_requests/models/ir_http.py b/monitoring_log_requests/models/ir_http.py index e71680a..03aa8a4 100644 --- a/monitoring_log_requests/models/ir_http.py +++ b/monitoring_log_requests/models/ir_http.py @@ -16,26 +16,30 @@ _logger = logging.getLogger('monitoring.http.requests') class IrHttp(models.AbstractModel): _inherit = 'ir.http' - def _dispatch(self): + @classmethod + def _dispatch(cls): begin = time.time() - response = super(IrHttp, self)._dispatch() + response = super(IrHttp, cls)._dispatch() end = time.time() - if (not self._monitoring_blacklist(http_request) and - self._monitoring_filter(http_request)): - info = self._monitoring_info(http_request, response, begin, end) - self._monitoring_log(info) + if (not cls._monitoring_blacklist(http_request) and + cls._monitoring_filter(http_request)): + info = cls._monitoring_info(http_request, response, begin, end) + cls._monitoring_log(info) return response - def _monitoring_blacklist(self, request): + @classmethod + def _monitoring_blacklist(cls, request): path_info = request.httprequest.environ.get('PATH_INFO') if path_info.startswith('/longpolling/'): return True return False - def _monitoring_filter(self, request): + @classmethod + def _monitoring_filter(cls, _): return True - def _monitoring_info(self, request, response, begin, end): + @classmethod + def _monitoring_info(cls, request, response, begin, end): path = request.httprequest.environ.get('PATH_INFO') info = { # timing @@ -74,5 +78,6 @@ class IrHttp(models.AbstractModel): }) return info - def _monitoring_log(self, info): + @classmethod + def _monitoring_log(cls, info): _logger.info(json.dumps(info))