advanced logging and image path fix implemented

app.py

@@ -3,6 +3,8 @@ from deepface import DeepFace
 from pydantic import BaseModel
 from fastapi.responses import JSONResponse
 import base64
+import json
+import logging
 import cv2
 import numpy as np
 import requests
@@ -13,6 +15,14 @@ class Verify(BaseModel):
     images: list[str]
 
 app = FastAPI()
+logger = logging.getLogger("plugg_verification")
+if not logger.handlers:
+    logging.basicConfig(level=logging.INFO)
+
+
+def log_event(event_type: str, **fields):
+    payload = {"event": event_type, **fields}
+    logger.error(json.dumps(payload, default=str))
 
 def load_image(source: str):
     """
@@ -60,6 +70,7 @@ def verify(v: Verify):
         selfie_img = load_image(selfie)
     except Exception as e:
         print(f"Failed to load selfie image: {e}")
+        log_event("load_error", target="selfie", source=selfie, error=str(e))
         return JSONResponse(content={"verified": False, "image": None, "error": "failed_to_load_selfie"})
 
     for image in gallery:
@@ -68,12 +79,15 @@ def verify(v: Verify):
             gallery_img = load_image(image)
         except Exception as e:
             print(f"Failed to load gallery image {image}: {e}")
+            log_event("load_error", target="gallery", source=image, error=str(e))
             continue
         try:
+            selfie_candidate = selfie_img.copy()
+            gallery_candidate = gallery_img.copy()
             # Use a more robust detector; fall back if detection fails
             result = DeepFace.verify(
-                img1_path=selfie_img,
-                img2_path=gallery_img,
+                img1_path=selfie_candidate,
+                img2_path=gallery_candidate,
                 detector_backend="retinaface"
             )
             if result.get("verified", False):
@@ -83,12 +97,31 @@ def verify(v: Verify):
         except Exception as e:
             msg = str(e)
             print(f"DeepFace verification error for {image}: {msg}")
+            # Retry once when DeepFace specifically fails while processing img1_path.
+            # Re-load selfie to avoid potential in-place mutation side effects.
+            if "img1_path" in msg:
+                try:
+                    selfie_retry = load_image(selfie)
+                    result = DeepFace.verify(
+                        img1_path=selfie_retry,
+                        img2_path=gallery_img.copy(),
+                        detector_backend="retinaface"
+                    )
+                    if result.get("verified", False):
+                        true_count += 1
+                        if true_count >= 2:
+                            return JSONResponse(content={"verified": True, "image": image})
+                    continue
+                except Exception as e_retry:
+                    print(f"DeepFace img1_path retry error for {image}: {e_retry}")
+                    log_event("img1_path_error", gallery_image=image, error=str(e_retry))
             # Optional fallback without strict detection to avoid generic img2_path errors
             if "Face could not be detected" in msg or "No face" in msg:
+                log_event("face_not_detected", gallery_image=image, error=msg)
                 try:
                     result = DeepFace.verify(
-                        img1_path=selfie_img,
-                        img2_path=gallery_img,
+                        img1_path=selfie_img.copy(),
+                        img2_path=gallery_img.copy(),
                         detector_backend="retinaface",
                         enforce_detection=False
                     )
@@ -98,4 +131,4 @@ def verify(v: Verify):
                             return JSONResponse(content={"verified": True, "image": image})
                 except Exception as e2:
                     print(f"DeepFace fallback error for {image}: {e2}")
-    return JSONResponse(content={"verified": False, "image": None})
+    return JSONResponse(content={"verified": False, "image": None})