clean up the dormant PII logs (#312)

* clean up the dormant PII logs

* lint fix

Author: Karthikeyannhs

src/eligibility_signposting_api/repos/person_repo.py

@@ -38,14 +38,11 @@ def __init__(self, table: Annotated[Any, Inject(qualifier="person_table")]) -> N
 
     def get_eligibility_data(self, nhs_number: NHSNumber) -> Person:
         response = self.table.query(KeyConditionExpression=Key("NHS_NUMBER").eq(nhs_number))
-        logger.debug("response %r for %r", response, nhs_number, extra={"response": response, "nhs_number": nhs_number})
 
         if not (items := response.get("Items")) or not next(
             (item for item in items if item.get("ATTRIBUTE_TYPE") == "PERSON"), None
         ):
             message = f"Person not found with nhs_number {nhs_number}"
             raise NotFoundError(message)
 
-        logger.debug("returning items %s", items, extra={"items": items})
-
         return Person(data=items)

src/eligibility_signposting_api/services/eligibility_services.py

@@ -42,15 +42,6 @@ def get_eligibility_status(
             try:
                 person_data = self.person_repo.get_eligibility_data(nhs_number)
                 campaign_configs = list(self.campaign_repo.get_campaign_configs())
-                logger.debug(
-                    "got person_data for %r",
-                    nhs_number,
-                    extra={
-                        "campaign_configs": [c.model_dump(by_alias=True) for c in campaign_configs],
-                        "person_data": person_data,
-                        "nhs_number": nhs_number,
-                    },
-                )
             except NotFoundError as e:
                 raise UnknownPersonError from e
             else:

tests/integration/lambda/test_app_running_as_lambda.py

@@ -81,7 +81,7 @@ def test_install_and_call_lambda_flask(
         has_entries(statusCode=HTTPStatus.OK, body=is_json_that(has_key("processedSuggestions"))),
     )
 
-    assert_that(log_output, contains_string("person_data"))
+    assert_that(log_output, contains_string("checking nhs_number"))
 
 
 def test_install_and_call_flask_lambda_over_http(