From c2d85f30140ea522fe5bfb50a77a32828a6e66c5 Mon Sep 17 00:00:00 2001 From: Guewen Baconnier Date: Tue, 29 Oct 2019 17:31:40 +0100 Subject: [PATCH] Revert "[11.0] [IMP] monitoring_status: check the db is reachable" --- monitoring_status/controllers/main.py | 21 +-------------------- 1 file changed, 1 insertion(+), 20 deletions(-) diff --git a/monitoring_status/controllers/main.py b/monitoring_status/controllers/main.py index 426a7cc..173b423 100644 --- a/monitoring_status/controllers/main.py +++ b/monitoring_status/controllers/main.py @@ -5,12 +5,9 @@ import logging import json -import psycopg2 - import werkzeug from odoo import http -from odoo.http import request from odoo.addons.web.controllers.main import ensure_db @@ -34,24 +31,10 @@ class Monitoring(http.Controller): @http.route('/monitoring/status', type='http', auth='none') def status(self): ensure_db() - http_status = 200 # TODO: add 'sub-systems' status and infos: # queue job, cron, database, ... headers = {'Content-Type': 'application/json'} info = {'status': 1} - # check the database connection - try: - cr = request.env.cr - cr.execute( - 'SELECT value ' - 'FROM ir_config_parameter ' - 'WHERE key=%s', - ('web.base.url',)) - result = cr.fetchone() - info['web.base.url'] = result or '' - except psycopg2.OperationalError as exc: - info['database_error'] = str(exc) - http_status = 503 session = http.request.session # We set a custom expiration of 1 second for this request, as we do a # lot of health checks, we don't want those anonymous sessions to be @@ -61,6 +44,4 @@ class Monitoring(http.Controller): # Redis. if not session.uid: session.expiration = 1 - return werkzeug.wrappers.Response( - json.dumps(info), status=http_status, headers=headers - ) + return werkzeug.wrappers.Response(json.dumps(info), headers=headers)