fix: Improve logging

Author: tmathern

src/c2pa/c2pa.py

@@ -865,6 +865,7 @@ def sign_file(
             try:
                 os.remove(dest_path)
             except OSError:
+                logger.warning("Failed to remove destination file")
                 pass  # Ignore cleanup errors
 
         # Re-raise the error
@@ -1116,8 +1117,8 @@ def __del__(self):
                     try:
                         _lib.c2pa_release_stream(self._stream)
                     except Exception:
-                        # Destructors shouldn't raise exceptions, just log
-                        # silently
+                        # Destructors shouldn't raise exceptions
+                        logger.warning("Failed to release Stream")
                         pass
                     finally:
                         self._stream = None
@@ -1499,6 +1500,9 @@ def _cleanup_resources(self):
                         _lib.c2pa_reader_free(self._reader)
                     except Exception:
                         # Cleanup failure doesn't raise exceptions
+                        logger.warning(
+                            "Failed to free native Reader resources"
+                        )
                         pass
                     finally:
                         self._reader = None
@@ -1509,16 +1513,18 @@ def _cleanup_resources(self):
                         self._own_stream.close()
                     except Exception:
                         # Cleanup failure doesn't raise exceptions
+                        logger.warning("Failed to close Reader stream")
                         pass
                     finally:
                         self._own_stream = None
 
-                # Clean up backing file
+                # Clean up backing file (if needed)
                 if self._backing_file:
                     try:
                         self._backing_file.close()
                     except Exception:
                         # Cleanup failure doesn't raise exceptions
+                        logger.warning("Failed to close Reader backing file")
                         pass
                     finally:
                         self._backing_file = None
@@ -1819,6 +1825,9 @@ def _cleanup_resources(self):
                         _lib.c2pa_signer_free(self._signer)
                     except Exception:
                         # Cleanup failure doesn't raise exceptions
+                        logger.warning(
+                            "Failed to free C2PA Signer during cleanup"
+                        )
                         pass
                     finally:
                         self._signer = None
@@ -2372,6 +2381,9 @@ def _sign_internal(
                     _lib.c2pa_manifest_bytes_free(manifest_bytes_ptr)
                 except Exception:
                     # Ignore errors during cleanup
+                    logger.warning(
+                        "Failed to release native manifest bytes memory"
+                    )
                     pass
 
         return manifest_bytes
@@ -2483,6 +2495,9 @@ def _cleanup_resources(self):
                         _lib.c2pa_builder_free(self._builder)
                     except Exception:
                         # Log cleanup errors but don't raise exceptions
+                        logger.warning(
+                            "Failed to release native Builder resources"
+                        )
                         pass
                     finally:
                         # Always clear the pointer and mark as closed