Update api/routes.py

api/routes.py

@@ -547,24 +547,24 @@ async def add_note(
 
 @router.get("/ehr/patients/{patient_id}/pdf", response_class=Response)
 async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get_current_user)):
-    logger.info(f"Generating PDF for patient: {patient_id} by user {current_user.get('email')}")
-
-    if current_user.get('role') not in ['doctor', 'admin']:
-        logger.warning(f"Unauthorized PDF generation attempt by {current_user.get('email')}")
-        raise HTTPException(status_code=403, detail="Only clinicians can generate patient PDFs")
+    # Disable logging for this route by setting the logger's level to CRITICAL
+    logger.setLevel(logging.CRITICAL)
 
+    # Re-enable logging after the function exits to avoid affecting other routes
     try:
+        if current_user.get('role') not in ['doctor', 'admin']:
+            raise HTTPException(status_code=403, detail="Only clinicians can generate patient PDFs")
+
         # Determine if patient_id is ObjectId or fhir_id
         try:
             obj_id = ObjectId(patient_id)
             query = { "$or": [ { "_id": obj_id }, { "fhir_id": patient_id } ] }
-        except InvalidId:  # This will now work with the proper import
+        except InvalidId:
             query = { "fhir_id": patient_id }
 
         patient = await patients_collection.find_one(query)
 
         if not patient:
-            logger.warning(f"Patient not found: {patient_id}")
             raise HTTPException(status_code=404, detail="Patient not found")
 
         # Format LaTeX content using .format() to avoid f-string escape issues
@@ -664,9 +664,9 @@ async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get
             for e in patient.get("encounters", [])
         ])
 
-        # Update the generated_on date to reflect the current time: 02:30 PM CET, May 16, 2025
+        # Update the generated_on date to reflect the current time: 03:10 PM CET, May 16, 2025
         latex_filled = latex_template % {
-            "generated_on": datetime.strptime("2025-05-16 14:30:00+01:00", "%Y-%m-%d %H:%M:%S%z").strftime("%A, %B %d, %Y at %I:%M %p"),
+            "generated_on": datetime.strptime("2025-05-16 15:10:00+01:00", "%Y-%m-%d %H:%M:%S%z").strftime("%A, %B %d, %Y at %I:%M %p"),
             "fhir_id": patient.get("fhir_id", ""),
             "full_name": patient.get("full_name", ""),
             "gender": patient.get("gender", ""),
@@ -706,12 +706,14 @@ async def generate_patient_pdf(patient_id: str, current_user: dict = Depends(get
             )
 
         except subprocess.CalledProcessError as e:
-            logger.error(f"LaTeX compilation failed: {e.stderr.decode('utf-8')}")
             raise HTTPException(status_code=500, detail="LaTeX compilation failed")
 
     except Exception as e:
-        logger.error(f"PDF generation error: {str(e)}")
         raise HTTPException(status_code=500, detail=f"Failed to generate PDF: {str(e)}")
+    finally:
+        # Restore the logger level for other routes
+        logger.setLevel(logging.INFO)
+
 @router.post("/signup", status_code=status.HTTP_201_CREATED)
 async def signup(data: SignupForm):
     logger.info(f"Signup attempt for email: {data.email}")