From c0f5e75040af84d79923624f6de4af343ca133c6 Mon Sep 17 00:00:00 2001
From: Ashutosh singh <55102089+Ashutosh619-sudo@users.noreply.github.com>
Date: Mon, 14 Oct 2024 14:28:05 +0530
Subject: [PATCH] feat: Adding extensive loggers (#224)

---
 sage_desktop_sdk/core/client.py | 12 ++++++++++++
 1 file changed, 12 insertions(+)

diff --git a/sage_desktop_sdk/core/client.py b/sage_desktop_sdk/core/client.py
index a60f390..4212a9e 100644
--- a/sage_desktop_sdk/core/client.py
+++ b/sage_desktop_sdk/core/client.py
@@ -1,4 +1,5 @@
 
+import logging
 import requests
 import json
 from typing import List, Dict, Generator
@@ -14,6 +15,7 @@
     InternalServerError
 )
 
+logger = logging.getLogger(__name__)
 
 class Client:
     """
@@ -120,6 +122,8 @@ def _query_get_all_generator(self, url: str, is_paginated: bool = False) -> Gene
 
                 data = json.loads(response.text)
 
+                logger.debug('Response for get request for url: %s, %s', request_url, response.text)
+
                 if not data:
                     break
 
@@ -131,6 +135,7 @@ def _query_get_all_generator(self, url: str, is_paginated: bool = False) -> Gene
                     break
 
             except requests.exceptions.HTTPError as err:
+                logger.info('Response for get request for url: %s, %s', url, err.response.text)
                 if err.response.status_code == 400:
                     raise WrongParamsError('Some of the parameters are wrong', response.text)
 
@@ -162,9 +167,11 @@ def _query_get_all(self, url: str) -> List[Dict]:
         response = requests.get(url=request_url, headers=api_headers)
 
         if response.status_code == 200:
+            logger.debug('Response for get request for url: %s, %s', request_url, response.text)
             data = json.loads(response.text)
             return data
 
+        logger.info('Response for get request for url: %s, %s', url, response.text)
         if response.status_code == 400:
             raise WrongParamsError('Some of the parameters are wrong', response.text)
 
@@ -196,9 +203,11 @@ def _query_get_by_id(self, url: str) -> List[Dict]:
         response = requests.get(url=request_url, headers=api_headers)
 
         if response.status_code == 200:
+            logger.debug('Response for get request for url: %s, %s', request_url, response.text)
             data = json.loads(response.text)
             return data
 
+        logger.info('Response for get request for url: %s, %s', url, response.text)
         if response.status_code == 400:
             raise WrongParamsError('Some of the parameters are wrong', response.text)
 
@@ -228,10 +237,13 @@ def _post_request(self, url: str, data=None) -> Dict:
         }
 
         response = requests.post(url=request_url, headers=api_headers, data=data)
+        logger.debug('Payload for post request: %s', data)
 
         if response.status_code == 200:
+            logger.debug('Response for post request: %s', response.text)
             return json.loads(response.text)
 
+        logger.info('Response for post request: %s', response.text)
         if response.status_code == 400:
             raise WrongParamsError('Some of the parameters are wrong', json.loads(response.text))