From 7e298244527bcaaf175fb016ea199bc65b8da927 Mon Sep 17 00:00:00 2001 From: Sergi Delgado Segura Date: Thu, 13 Feb 2020 16:51:44 +0100 Subject: [PATCH] Improves API log and adds logs for get_appointment --- pisa/api.py | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/pisa/api.py b/pisa/api.py index e31d0d2..bf6665c 100644 --- a/pisa/api.py +++ b/pisa/api.py @@ -39,7 +39,7 @@ class API: remote_addr = request.environ.get("REMOTE_ADDR") remote_port = request.environ.get("REMOTE_PORT") - logger.info("Connection accepted", from_addr_port="{}:{}".format(remote_addr, remote_port)) + logger.info("Received add_appointment request", from_addr_port="{}:{}".format(remote_addr, remote_port)) # Check content type once if properly defined request_data = json.loads(request.get_json()) @@ -101,10 +101,16 @@ class API: - Appointments hold by the :obj:`Responder ` are flagged as ``dispute_triggered``. - Unknown appointments are flagged as ``not_found``. """ + remote_addr = request.environ.get("REMOTE_ADDR") + remote_port = request.environ.get("REMOTE_PORT") locator = request.args.get("locator") response = [] + logger.info( + "Received get_appointment request", from_addr_port="{}:{}".format(remote_addr, remote_port), locator=locator + ) + # ToDo: #15-add-system-monitor if not isinstance(locator, str) or len(locator) != LOCATOR_LEN_HEX: response.append({"locator": locator, "status": "not_found"})