Remove azure-storage-logging

- Update azure-* dependencies using django-storages
- Add azure-identity to support azure identity (not used right now)

Author: thenav56

api/logger.py

@@ -1,11 +1,6 @@
 import logging
 import sys
 
-from azure_storage_logging.handlers import (
-    BlobStorageTimedRotatingFileHandler as storage,
-)
-from django.conf import settings
-
 formatter = logging.Formatter(fmt="%(asctime)s %(levelname)-8s %(message)s", datefmt="%Y-%m-%d %H:%M:%S")
 
 screen_handler = logging.StreamHandler(stream=sys.stdout)
@@ -15,17 +10,25 @@
 logger.setLevel("DEBUG")
 logger.addHandler(screen_handler)
 
-if settings.AZURE_STORAGE_ACCOUNT is not None and settings.AZURE_STORAGE_KEY is not None:
-    handler = storage(
-        account_name=settings.AZURE_STORAGE_ACCOUNT,
-        account_key=settings.AZURE_STORAGE_KEY,
-        filename="go.log",
-        when="M",
-        interval=90,
-        container="logs",
-        encoding="utf-8",
-    )
-    handler.setFormatter(formatter)
-    logger.addHandler(handler)
-else:
-    logger.warning("No Azure credentials found, falling back to local logs.")
+# NOTE: Disabled due to old azure_storage_logging package.
+# TODO: Replace this with opentelemetry?
+#  https://learn.microsoft.com/en-us/azure/azure-monitor/app/opentelemetry-enable?tabs=python
+# from azure_storage_logging.handlers import (
+#     BlobStorageTimedRotatingFileHandler as storage,
+# )
+# from django.conf import settings
+#
+# if settings.AZURE_STORAGE_ACCOUNT is not None and settings.AZURE_STORAGE_KEY is not None:
+#     handler = storage(
+#         account_name=settings.AZURE_STORAGE_ACCOUNT,
+#         account_key=settings.AZURE_STORAGE_KEY,
+#         filename="go.log",
+#         when="M",
+#         interval=90,
+#         container="logs",
+#         encoding="utf-8",
+#     )
+#     handler.setFormatter(formatter)
+#     logger.addHandler(handler)
+# else:
+#     logger.warning("No Azure credentials found, falling back to local logs.")

pyproject.toml

@@ -12,13 +12,6 @@ dependencies = [
     "Pillow==10.3.0",
     "PyPDF2==1.27.9",
     "pdf2image==1.16.0",
-    "azure-storage==0.36.0",
-    "azure-common==1.1.19",
-    "azure-nspkg==3.0.2",
-    "azure-storage-blob==1.5.0",
-    "azure-storage-common==1.4.0",
-    "azure-storage-logging==0.5.1",
-    "azure-storage-nspkg==3.1.0",
     "beautifulsoup4==4.6.3",
     "boto3==1.20.38",
     "choicesenum==0.7.0",
@@ -39,7 +32,7 @@ dependencies = [
     "django-read-only==1.12.0",
     "django-reversion-compare==0.16.2",
     "django-reversion==5.0.12",
-    "django-storages[s3]==1.11.1",  # FIXME: Add azure
+    "django-storages[s3,azure]==1.14.5",
     "django-tinymce==4.1.0",
     "django-oauth-toolkit==3.0.1",
     "djangorestframework-csv==2.1.1",
@@ -85,7 +78,7 @@ dependencies = [
     "pyjwt",
     "shapely",
     "colorlog",
-    # "azure-identity",  # Required by django-storages[azure] if used
+    "azure-identity",  # Required by django-storages[azure] if used
     "mapbox-tilesets",
     "ipython",
     "tiktoken",