srx-lib-azure 0.1.4__tar.gz → 0.1.6__tar.gz

{srx_lib_azure-0.1.4 → srx_lib_azure-0.1.6}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: srx-lib-azure
-Version: 0.1.4
+Version: 0.1.6
 Summary: Azure helpers for SRX services: Blob, Email, Table
 Author-email: SRX <dev@srx.id>
 Requires-Python: >=3.12

{srx_lib_azure-0.1.4 → srx_lib_azure-0.1.6}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "srx-lib-azure"
-version = "0.1.4"
+version = "0.1.6"
 description = "Azure helpers for SRX services: Blob, Email, Table"
 readme = "README.md"
 requires-python = ">=3.12"

{srx_lib_azure-0.1.4 → srx_lib_azure-0.1.6}/src/srx_lib_azure/blob.py

@@ -10,18 +10,32 @@ from loguru import logger
 
 
 class AzureBlobService:
-    """Minimal Azure Blob helper with SAS URL generation."""
-
-    def __init__(self) -> None:
-        self.container_name = os.getenv("AZURE_BLOB_CONTAINER", "uploads")
-        self.connection_string = os.getenv("AZURE_STORAGE_CONNECTION_STRING")
-        self.account_key = os.getenv("AZURE_STORAGE_ACCOUNT_KEY")
-        self.sas_token = os.getenv("AZURE_SAS_TOKEN")
-        self.base_blob_url = os.getenv("AZURE_BLOB_URL")
-
-        if not self.connection_string:
+    """Minimal Azure Blob helper with SAS URL generation.
+
+    All configuration can be passed explicitly via constructor. If omitted, falls back
+    to environment variables. By default, it does not warn at startup when not
+    configured; operations will error if required values are missing.
+    """
+
+    def __init__(
+        self,
+        *,
+        connection_string: Optional[str] = None,
+        account_key: Optional[str] = None,
+        container_name: Optional[str] = None,
+        base_blob_url: Optional[str] = None,
+        sas_token: Optional[str] = None,
+        warn_if_unconfigured: bool = False,
+    ) -> None:
+        self.container_name = container_name or os.getenv("AZURE_BLOB_CONTAINER", "uploads")
+        self.connection_string = connection_string or os.getenv("AZURE_STORAGE_CONNECTION_STRING")
+        self.account_key = account_key or os.getenv("AZURE_STORAGE_ACCOUNT_KEY")
+        self.sas_token = sas_token or os.getenv("AZURE_SAS_TOKEN")
+        self.base_blob_url = base_blob_url or os.getenv("AZURE_BLOB_URL")
+
+        if warn_if_unconfigured and not self.connection_string:
             logger.warning(
-                "Azure Storage connection string not configured; blob operations will fail."
+                "Azure Storage connection string not configured; blob operations may fail."
             )
 
     def _get_blob_service(self) -> BlobServiceClient:

srx_lib_azure-0.1.6/src/srx_lib_azure/email.py

@@ -0,0 +1,66 @@
+import os
+import logging
+from typing import Dict, Any
+
+try:
+    from azure.communication.email.aio import EmailClient
+except Exception:  # pragma: no cover - optional dependency at import time
+    EmailClient = None  # type: ignore
+
+logger = logging.getLogger(__name__)
+
+
+class EmailService:
+    """Thin wrapper over Azure Communication Services EmailClient.
+
+    Does not raise on missing configuration to keep the library optional.
+    If not configured, send calls are skipped with a warning and a 'skipped' status.
+    """
+
+    def __init__(
+        self,
+        *,
+        connection_string: str | None = None,
+        sender_address: str | None = None,
+        warn_if_unconfigured: bool = False,
+    ):
+        self.connection_string = connection_string or os.getenv("ACS_CONNECTION_STRING")
+        self.sender_address = sender_address or os.getenv("EMAIL_SENDER")
+        if not self.connection_string or not self.sender_address or EmailClient is None:
+            self.email_client = None
+            if warn_if_unconfigured:
+                logger.warning(
+                    "EmailService not configured (missing ACS_CONNECTION_STRING/EMAIL_SENDER or azure SDK). Calls will be skipped."
+                )
+        else:
+            try:
+                self.email_client = EmailClient.from_connection_string(self.connection_string)
+            except Exception as e:
+                self.email_client = None
+                logger.warning("EmailService initialization failed: %s", e)
+
+    async def send_notification(self, recipient: str, subject: str, body: str, html: bool = False) -> Dict[str, Any]:
+        if not self.email_client or not self.sender_address:
+            logger.warning("Email skipped: service not configured")
+            return {"status": "skipped", "message": "Email service not configured"}
+        message = {
+            "content": {"subject": subject},
+            "recipients": {"to": [{"address": recipient}]},
+            "senderAddress": self.sender_address,
+        }
+        if html:
+            message["content"]["html"] = body
+        else:
+            message["content"]["plainText"] = body
+        try:
+            poller = await self.email_client.begin_send(message)
+            result = await poller.result()
+            message_id = result.get("id")
+            if message_id:
+                logger.info("Email sent to %s with Message ID: %s", recipient, message_id)
+                return {"status": "success", "message": "Email sent successfully", "message_id": message_id}
+            logger.error("Failed to send email. Result: %s", result)
+            return {"status": "error", "message": f"Failed to send email: {result}"}
+        except Exception as e:
+            logger.error("Email send exception: %s", e)
+            return {"status": "error", "message": str(e)}

{srx_lib_azure-0.1.4 → srx_lib_azure-0.1.6}/src/srx_lib_azure/table.py

@@ -19,7 +19,11 @@ def _now_iso() -> str:
 
 @dataclass
 class AzureTableService:
-    connection_string: Optional[str] = os.getenv("AZURE_STORAGE_CONNECTION_STRING")
+    connection_string: Optional[str] = None
+
+    def __init__(self, connection_string: Optional[str] = None) -> None:
+        # Constructor injection preferred; fallback to env only if not provided
+        self.connection_string = connection_string or os.getenv("AZURE_STORAGE_CONNECTION_STRING")
 
     def _get_client(self) -> "TableServiceClient":
         if not self.connection_string:
@@ -76,4 +80,3 @@ class AzureTableService:
         table = client.get_table_client(table_name)
         for entity in table.query_entities(filter=filter_query):
             yield dict(entity)
-

srx_lib_azure-0.1.4/src/srx_lib_azure/email.py

@@ -1,41 +0,0 @@
-import os
-from azure.communication.email.aio import EmailClient
-
-import logging
-
-logger = logging.getLogger(__name__)
-
-
-class EmailService:
-    """Thin wrapper over Azure Communication Services EmailClient."""
-
-    def __init__(self):
-        self.connection_string = os.getenv("ACS_CONNECTION_STRING")
-        self.sender_address = os.getenv("EMAIL_SENDER")
-        if not self.connection_string or not self.sender_address:
-            raise ValueError("Missing ACS_CONNECTION_STRING or EMAIL_SENDER in environment variables")
-        self.email_client = EmailClient.from_connection_string(self.connection_string)
-
-    async def send_notification(self, recipient: str, subject: str, body: str, html: bool = False):
-        message = {
-            "content": {"subject": subject},
-            "recipients": {"to": [{"address": recipient}]},
-            "senderAddress": self.sender_address,
-        }
-        if html:
-            message["content"]["html"] = body
-        else:
-            message["content"]["plainText"] = body
-        try:
-            poller = await self.email_client.begin_send(message)
-            result = await poller.result()
-            message_id = result.get("id")
-            if message_id:
-                logger.info("Email sent to %s with Message ID: %s", recipient, message_id)
-                return {"status": "success", "message": "Email sent successfully", "message_id": message_id}
-            logger.error("Failed to send email. Result: %s", result)
-            return {"status": "error", "message": f"Failed to send email: {result}"}
-        except Exception as e:
-            logger.error("Email send exception: %s", e)
-            return {"status": "error", "message": str(e)}
-