Merge pull request #14 from SamhammerAG/KIT-4148

KIT-4148 unify logging field "request_brand" to "brand"

Author: oschulzSam

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "sag-py-auth-brand"
-version = "1.2.1"
+version = "1.2.2"
 description = "Keycloak brand/instance authentication for python projects"
 authors = ["Samhammer AG"]
 license = "MIT"

sag_py_auth_brand/models.py

@@ -12,4 +12,4 @@ def __init__(self, issuer: str, audience: str, instance: str, stage: str) -> Non
 
 class BrandLogRecord(LogRecord):
     # The original brand of the request
-    request_brand: str
+    brand: str

sag_py_auth_brand/request_brand_logging_filter.py

@@ -11,6 +11,6 @@ def __init__(self, name: str = "") -> None:
 
     def filter(self, record: LogRecord) -> bool:
         if request_brand := get_request_brand():
-            record.request_brand = request_brand
+            record.brand = request_brand
 
         return True

setup.py

@@ -11,7 +11,7 @@
 
 setuptools.setup(
     name="sag-py-auth-brand",
-    version="1.2.1",
+    version="1.2.2",
     description="Keycloak brand/instance authentication for python projects",
     long_description=LONG_DESCRIPTION,
     long_description_content_type="text/markdown",

tests/test_request_brand_logging_filter.py

@@ -18,7 +18,7 @@ def test__get_field_value__with_brand() -> None:
 
     # Assert
     assert log_entry.msg == "A test message"
-    assert cast(BrandLogRecord, log_entry).request_brand == "myBrand"
+    assert cast(BrandLogRecord, log_entry).brand == "myBrand"
 
 
 def test__get_field_value__without_data() -> None: