django-log-formatter-asim 1.1.0a1__py3-none-any.whl → 1.1.0a2__py3-none-any.whl

django_log_formatter_asim/__init__.py

@@ -1,8 +1,9 @@
 import json
 import logging
+import os
 from datetime import datetime
+from datetime import timezone
 from importlib.metadata import distribution
-import os
 
 import ddtrace
 from ddtrace.trace import tracer
@@ -70,7 +71,7 @@ class ASIMRootFormatter:
 
     def get_log_dict(self):
         record = self.record
-        log_time = datetime.utcfromtimestamp(record.created).isoformat()
+        log_time = datetime.fromtimestamp(record.created, timezone.utc).isoformat()
         log_dict = {
             # Event fields...
             "EventMessage": record.getMessage(),
@@ -200,14 +201,6 @@ class ASIMRequestFormatter(ASIMRootFormatter):
             http_user_agent = request.META.get("HTTP_USER_AGENT", None)
         return http_user_agent
 
-    def _get_ip_address(self, request):
-        # Import here as ipware uses settings
-        from ipware import get_client_ip
-
-        client_ip, is_routable = get_client_ip(request)
-        return client_ip or "Unknown"
-
-
 ASIM_FORMATTERS = {
     "root": ASIMRootFormatter,
     "django.request": ASIMRequestFormatter,

django_log_formatter_asim/events/authentication.py

@@ -12,6 +12,7 @@ from .common import Result
 from .common import Server
 from .common import Severity
 from .common import _default_severity
+from .common import _get_client_ip_address
 
 
 class AuthenticationEvent(str, Enum):
@@ -78,6 +79,7 @@ def log_authentication(
                         - Django Authentication systems current username
                         - Django Session middlewares Session Key
                         - Client IP address
+                        - URL requested by the client
                         - Server hostname
     :param event: What authentication action was attempted, either "Logon" or "Logoff"
     :param result: What outcome did the action have, either "Success", "Failure", "Partial", "NA"
@@ -126,13 +128,18 @@ def log_authentication(
 
     if "hostname" in server:
         log["DvcHostname"] = server["hostname"]
-    elif hasattr(request, "environ") and "SERVER_NAME" in request.environ:
-        log["DvcHostname"] = request.environ["SERVER_NAME"]
+    elif "SERVER_NAME" in request.META:
+        log["DvcHostname"] = request.META["SERVER_NAME"]
 
     if "ip_address" in client:
         log["SrcIpAddr"] = client["ip_address"]
-    elif hasattr(request, "environ") and "REMOTE_ADDR" in request.environ:
-        log["SrcIpAddr"] = request.environ.get("REMOTE_ADDR")
+    elif client_ip := _get_client_ip_address(request):
+        log["SrcIpAddr"] = client_ip
+
+    if "requested_url" in client:
+        log["TargetUrl"] = client["requested_url"]
+    elif "SERVER_NAME" in request.META:
+        log["TargetUrl"] = request.scheme + "://" + request.get_host() + request.get_full_path()
 
     if "role" in user:
         log["ActorUserType"] = user["role"]

django_log_formatter_asim/events/common.py

@@ -2,6 +2,8 @@ from enum import Enum
 from typing import Optional
 from typing import TypedDict
 
+from django.http import HttpRequest
+
 
 class Result(str, Enum):
     Success = "Success"
@@ -23,6 +25,8 @@ class Client(TypedDict):
     """Internet Protocol Address of the client making the Authentication
     event."""
     ip_address: Optional[str]
+    """URL requested by the client."""
+    requested_url: Optional[str]
 
 
 class Server(TypedDict):
@@ -40,3 +44,11 @@ class Server(TypedDict):
 
 def _default_severity(result: Result) -> Severity:
     return Severity.Informational if result == Result.Success else Severity.Medium
+
+
+def _get_client_ip_address(request: HttpRequest) -> Optional[str]:
+    # Import here as ipware uses settings
+    from ipware import get_client_ip
+
+    client_ip, _ = get_client_ip(request)
+    return client_ip

django_log_formatter_asim/events/file_activity.py

@@ -12,6 +12,7 @@ from .common import Result
 from .common import Server
 from .common import Severity
 from .common import _default_severity
+from .common import _get_client_ip_address
 
 
 class FileActivityEvent(str, Enum):
@@ -92,6 +93,7 @@ def log_file_activity(
                     from which the following data will be logged if available
                         - Django Authentication systems current username
                         - Client IP address
+                        - URL requested by the client
                         - Server hostname
     :param event: What File Event action was attempted, one of:
                         - FileAccessed
@@ -167,13 +169,18 @@ def log_file_activity(
 
     if "hostname" in server:
         log["DvcHostname"] = server["hostname"]
-    elif hasattr(request, "environ") and "SERVER_NAME" in request.environ:
-        log["DvcHostname"] = request.environ["SERVER_NAME"]
+    elif "SERVER_NAME" in request.META:
+        log["DvcHostname"] = request.META["SERVER_NAME"]
 
     if "ip_address" in client:
         log["SrcIpAddr"] = client["ip_address"]
-    elif hasattr(request, "environ") and "REMOTE_ADDR" in request.environ:
-        log["SrcIpAddr"] = request.environ.get("REMOTE_ADDR")
+    elif client_ip := _get_client_ip_address(request):
+        log["SrcIpAddr"] = client_ip
+
+    if "requested_url" in client:
+        log["TargetUrl"] = client["requested_url"]
+    elif "SERVER_NAME" in request.META:
+        log["TargetUrl"] = request.scheme + "://" + request.get_host() + request.get_full_path()
 
     if "username" in user:
         log["ActorUsername"] = user["username"]

{django_log_formatter_asim-1.1.0a1.dist-info → django_log_formatter_asim-1.1.0a2.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: django-log-formatter-asim
-Version: 1.1.0a1
+Version: 1.1.0a2
 Summary: Formats Django logs in ASIM format.
 License: MIT
 Author: Department for Business and Trade Platform Team
@@ -16,6 +16,7 @@ Classifier: Programming Language :: Python :: 3.13
 Requires-Dist: ddtrace (>=3.2.1,<4.0.0)
 Requires-Dist: django (>=3,<5) ; python_version == "3.9"
 Requires-Dist: django (>=3,<6) ; python_version >= "3.10" and python_version < "4"
+Requires-Dist: django-ipware (>=7.0.1,<8.0.0)
 Description-Content-Type: text/markdown
 
 # Django ASIM log formatter

django_log_formatter_asim-1.1.0a2.dist-info/RECORD

@@ -0,0 +1,9 @@
+django_log_formatter_asim/__init__.py,sha256=LDQQch2GKY6LRj-KOME393FCiwjisQ49ATWBEqAP4ZA,7684
+django_log_formatter_asim/events/__init__.py,sha256=th8AEFNM-J5lNlO-d8Lk465jXqplE3IoTwj4DlscwYo,92
+django_log_formatter_asim/events/authentication.py,sha256=OMAT4P8ZRmeEWngHTLvscGOXtRE8t39izZpdKNot2R8,6479
+django_log_formatter_asim/events/common.py,sha256=G6h4ftfrAzXBNd3EkhFQsMv5BdGW0XtdeBKJ43Ja70s,1360
+django_log_formatter_asim/events/file_activity.py,sha256=RPlzJm8TjdT3szx1wClULG29Sz9RNA16NQmUJwhxjTc,6978
+django_log_formatter_asim-1.1.0a2.dist-info/LICENSE,sha256=dP79lN73--7LMApnankTGLqDbImXg8iYFqWgnExGkGk,1090
+django_log_formatter_asim-1.1.0a2.dist-info/METADATA,sha256=fFVGcfHJ9wOQq19lqnulIV5PgvHc99m1Mt8aNuUSMS8,5581
+django_log_formatter_asim-1.1.0a2.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
+django_log_formatter_asim-1.1.0a2.dist-info/RECORD,,

django_log_formatter_asim-1.1.0a1.dist-info/RECORD

@@ -1,9 +0,0 @@
-django_log_formatter_asim/__init__.py,sha256=i-7HqYE5s3hjmHLh4TTzLpEdgr5N22msFEQw16Pe_EI,7867
-django_log_formatter_asim/events/__init__.py,sha256=th8AEFNM-J5lNlO-d8Lk465jXqplE3IoTwj4DlscwYo,92
-django_log_formatter_asim/events/authentication.py,sha256=Nkgv-c_pNhwS8C1QYj0M3zH_cz2QJQqopzcMRhTv7Ls,6242
-django_log_formatter_asim/events/common.py,sha256=4P3lb-rfxv_4Vf2DfW-Mi3Sq7rIWqF_tGK8mPPCUMco,1039
-django_log_formatter_asim/events/file_activity.py,sha256=bKDpZcoBUePTeKCDPqLToGc8CA85_aXTw1c8KXoRLq0,6741
-django_log_formatter_asim-1.1.0a1.dist-info/LICENSE,sha256=dP79lN73--7LMApnankTGLqDbImXg8iYFqWgnExGkGk,1090
-django_log_formatter_asim-1.1.0a1.dist-info/METADATA,sha256=Fz6KC4Ai4jr3yq-Q1PDCtl0cp9mGlor0buzMCmh9J9s,5535
-django_log_formatter_asim-1.1.0a1.dist-info/WHEEL,sha256=b4K_helf-jlQoXBBETfwnf4B04YC67LOev0jo4fX5m8,88
-django_log_formatter_asim-1.1.0a1.dist-info/RECORD,,