From bf049bf6f4ec22b520f5f8e63bd246d12326dd38 Mon Sep 17 00:00:00 2001 From: Giorgio Ravera Date: Fri, 1 May 2026 23:26:46 +0200 Subject: [PATCH] Updated json for aliases and hosts with details --- backend/routes/aliases.py | 62 ++++++++++++++++++++++++++++----------- backend/routes/hosts.py | 62 ++++++++++++++++++++++++++++----------- 2 files changed, 90 insertions(+), 34 deletions(-) diff --git a/backend/routes/aliases.py b/backend/routes/aliases.py index 66673fa..4720e2e 100644 --- a/backend/routes/aliases.py +++ b/backend/routes/aliases.py @@ -78,13 +78,17 @@ def api_get_alias(request: Request, alias_id: int): try: alias = get_alias(alias_id) if not alias: # None or empty dict + took_ms = (time.monotonic_ns() - start_ns) / 1_000_000 raise HTTPException( status_code=status.HTTP_404_NOT_FOUND, detail={ "code": "ALIAS_NOT_FOUND", "status": "failure", "message": "Alias not found", - "alias_id": alias_id, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) return alias @@ -101,7 +105,10 @@ def api_get_alias(request: Request, alias_id: int): "code": "ALIAS_GET_ERROR", "status": "failure", "message": "Internal error getting alias", - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) @@ -126,8 +133,10 @@ def api_add_alias(request: Request, data: dict): "code": "ALIAS_ADDED", "status": "success", "message": "Alias added successfully", - "alias_id": alias_id, - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, } # Already present @@ -138,7 +147,10 @@ def api_add_alias(request: Request, data: dict): "code": "ALIAS_ALREADY_PRESENT", "status": "failure", "message": "Alias already present", - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) @@ -154,7 +166,10 @@ def api_add_alias(request: Request, data: dict): "code": "ALIAS_ADD_ERROR", "status": "failure", "message": "Internal error adding alias", - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) @@ -179,8 +194,10 @@ def api_update_alias(request: Request, data: dict, alias_id: int): "code": "ALIAS_UPDATED", "status": "success", "message": "Alias updated successfully", - "alias_id": alias_id, - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, } # Not Found @@ -191,8 +208,10 @@ def api_update_alias(request: Request, data: dict, alias_id: int): "code": "ALIAS_NOT_FOUND", "status": "failure", "message": "Alias not found", - "alias_id": alias_id, - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) @@ -208,8 +227,10 @@ def api_update_alias(request: Request, data: dict, alias_id: int): "code": "ALIAS_UPDATE_ERROR", "status": "failure", "message": "Internal error updating alias", - "alias_id": alias_id, - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) @@ -234,7 +255,10 @@ def api_delete_alias(request: Request, alias_id: int): "code": "ALIAS_DELETED", "status": "success", "message": "Alias deleted successfully", - "details": {"took_ms": took_ms, "alias_id": alias_id,}, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, } # Not Found @@ -245,8 +269,10 @@ def api_delete_alias(request: Request, alias_id: int): "code": "ALIAS_NOT_FOUND", "status": "failure", "message": "Alias not found", - "alias_id": alias_id, - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) @@ -262,7 +288,9 @@ def api_delete_alias(request: Request, alias_id: int): "code": "ALIAS_DELETE_ERROR", "status": "failure", "message": "Internal error deleting alias", - "alias_id": alias_id, - "took_ms": took_ms, + "details": { + "alias_id": alias_id, + "took_ms": took_ms, + }, }, ) diff --git a/backend/routes/hosts.py b/backend/routes/hosts.py index 07362af..74585bb 100644 --- a/backend/routes/hosts.py +++ b/backend/routes/hosts.py @@ -78,13 +78,17 @@ def api_get_host(request: Request, host_id: int): try: host = get_host(host_id) if not host: # None or empty dict + took_ms = (time.monotonic_ns() - start_ns) / 1_000_000 raise HTTPException( status_code=status.HTTP_404_NOT_FOUND, detail={ "code": "HOST_NOT_FOUND", "status": "failure", "message": "Host not found", - "host_id": host_id, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) return host @@ -101,7 +105,10 @@ def api_get_host(request: Request, host_id: int): "code": "HOST_GET_ERROR", "status": "failure", "message": "Internal error getting host", - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) @@ -126,8 +133,10 @@ def api_add_host(request: Request, data: dict): "code": "HOST_ADDED", "status": "success", "message": "Host added successfully", - "host_id": host_id, - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, } # Already present @@ -138,7 +147,10 @@ def api_add_host(request: Request, data: dict): "code": "HOST_ALREADY_PRESENT", "status": "failure", "message": "Host already present", - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) @@ -154,7 +166,10 @@ def api_add_host(request: Request, data: dict): "code": "HOST_ADD_ERROR", "status": "failure", "message": "Internal error adding host", - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) @@ -179,8 +194,10 @@ def api_update_host(request: Request, data: dict, host_id: int): "code": "HOST_UPDATED", "status": "success", "message": "Host updated successfully", - "host_id": host_id, - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, } # Not Found @@ -191,8 +208,10 @@ def api_update_host(request: Request, data: dict, host_id: int): "code": "HOST_NOT_FOUND", "status": "failure", "message": "Host not found", - "host_id": host_id, - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) @@ -208,8 +227,10 @@ def api_update_host(request: Request, data: dict, host_id: int): "code": "HOST_UPDATE_ERROR", "status": "failure", "message": "Internal error updating host", - "host_id": host_id, - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) @@ -234,7 +255,10 @@ def api_delete_host(request: Request, host_id: int): "code": "HOST_DELETED", "status": "success", "message": "Host deleted successfully", - "details": {"took_ms": took_ms, "host_id": host_id,}, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, } # Not Found @@ -245,8 +269,10 @@ def api_delete_host(request: Request, host_id: int): "code": "HOST_NOT_FOUND", "status": "failure", "message": "Host not found", - "host_id": host_id, - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) @@ -262,7 +288,9 @@ def api_delete_host(request: Request, host_id: int): "code": "HOST_DELETE_ERROR", "status": "failure", "message": "Internal error deleting host", - "host_id": host_id, - "took_ms": took_ms, + "details": { + "host_id": host_id, + "took_ms": took_ms, + }, }, ) -- 2.47.3