Cleanup, improve comments

audit_log/headers.py

@@ -27,6 +27,8 @@ def parse_spiffe(xfcc_header: str) -> ParsedSPIFFE:
 
     Raises exception if header is invalid.
 
+    Note: When using this with Istio, the ingress gateway will also include a SPIFFE header. However, this does not represent the actual principal.
+
     Args:
         xfcc_header (str): X-Forwarded-Client-Cert header contents
 

audit_log/log.py

@@ -8,12 +8,6 @@
 from functools import singledispatch
 from typing import Any
 
-from audit_log.headers import (
-    ISS_HEADER,
-    SUB_HEADER,
-    SUB_TYPE_HEADER,
-    get_principal_from_headers,
-)
 from audit_log.schema import (
     SCHEMA_VERSION,
     ActionType,
@@ -80,21 +74,3 @@ def log(
             }
         )
     )
-
-
-if __name__ == "__main__":
-    headers = {
-        SUB_HEADER: "[email protected]",
-        ISS_HEADER: "respect_mime",
-        SUB_TYPE_HEADER: "USER",
-    }
-    principal = get_principal_from_headers(headers)
-    log(
-        action_type=ActionType.CREATE,
-        resource_type="test",
-        resource_id=uuid.uuid4(),
-        result=OutcomeResult.SUCCEEDED,
-        request_id=uuid.uuid4(),
-        outcome_reason=ValueError("test"),  # type: ignore[arg-type] # Purposeful mis-type to verify
-        principal=principal,
-    )