refactor

Author: vvillait88

src/peopledatalabs/requests.py

@@ -47,6 +47,18 @@ def __post_init__(self):
         self.params = self.validator(**self.params).dict(exclude_none=True)
         logger.debug("Request object params after validation: %s", self.params)
 
+    def _sanitize_params(self) -> dict:
+        """
+        Creates a copy of params with sensitive data redacted for logging.
+
+        Returns:
+            A sanitized copy of the parameters dictionary.
+        """
+        logger_params = self.params.copy()
+        if "api_key" in logger_params:
+            logger_params["api_key"] = "***"
+        return logger_params
+
     def get(self):
         """
         Executes a GET request from the specified API.
@@ -57,13 +69,10 @@ def get(self):
             A requests.Response object with the result of the HTTP call.
         """
         self.params["api_key"] = self.api_key
-        logger_params = self.params.copy()
-        if "api_key" in logger_params:
-            logger_params["api_key"] = "***"
         logger.info(
             "Calling %s with params: %s",
             self.url,
-            json.dumps(logger_params, indent=2),
+            json.dumps(self._sanitize_params(), indent=2),
         )
         return requests.get(
             self.url, params=self.params, headers=self.headers, timeout=None
@@ -79,13 +88,10 @@ def post(self):
             A requests.Response object with the result of the HTTP call.
         """
         self.headers["X-api-key"] = self.api_key
-        logger_params = self.params.copy()
-        if "api_key" in logger_params:
-            logger_params["api_key"] = "***"
         logger.info(
             "Calling %s with params: %s",
             self.url,
-            json.dumps(logger_params, indent=2),
+            json.dumps(self._sanitize_params(), indent=2),
         )
         return requests.post(
             self.url, json=self.params, headers=self.headers, timeout=None