refactor: log that shows amount of not recieved DOIs from BASE

Author: andreishket

server/workers/orcid/src/orcid_service.py

@@ -203,10 +203,7 @@ def request_base_metadata(self, dois: List[str], params: Dict[str, str]) -> pd.D
             base_response: str = get_nested_value(result, ["input_data", "metadata"], '[]') # type: ignore
 
             batch_df = pd.DataFrame(json.loads(base_response))
-            if len(batch_df) < len(batch):
-                self.logger.warning(
-                    f"BASE response shortfall: requested {len(batch)} DOIs, received {len(batch_df)} rows (limit or missing records?)"
-                )
+            self._log_is_base_response_missing_dois(batch, batch_df)
 
             base_metadata = pd.concat([
                 base_metadata,
@@ -679,4 +676,11 @@ def _handle_insufficient_results(self, params: Dict[str, str], orcid_id: str) ->
     def _handle_error(self, params: Dict[str, str], reason: str, exception: Exception) -> ErrorResult:
         self.logger.debug(f"Error processing ORCID: {exception}. Params: {params}")
         self.logger.debug(exception.__traceback__)
-        return {"status": "error", "reason": [reason]}
+        return {"status": "error", "reason": [reason]}
+
+    def _log_is_base_response_missing_dois(self, batch: List[str], batch_df: pd.DataFrame):
+        is_some_dois_missing = len(batch_df) < len(batch)
+        if is_some_dois_missing:
+            self.logger.debug(
+                f"BASE response statistics: requested {len(batch)} DOIs, received {len(batch_df)} rows"
+            )