Merge pull request #345 from bthorben/master

Better logging for _get_attribute_value to debug wrong configurations
IdentityPython · Sep 20, 2022 · 16bb169 · 16bb169
2 parents bbe845a + 455052f
commit 16bb169
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/djangosaml2/backends.py b/djangosaml2/backends.py
@@ -110,9 +110,11 @@ def _get_attribute_value(
             return saml_attribute[0]
         else:
             logger.error(
-                "attributes[saml_attr] attribute "
-                "value is missing. Probably the user "
-                "session is expired."
+                "attributes[saml_attr] attribute value is missing. "
+                f"Either the user session is expired or your mapping is invalid.\n"
+                f"django_field: {django_field}\n"
+                f"attributes: {attributes}\n"
+                f"attribute_mapping: {attribute_mapping}"
             )
 
     def authenticate(

diff --git a/setup.py b/setup.py
@@ -27,7 +27,7 @@ def read(*rnames):
 
 setup(
     name="djangosaml2",
-    version="1.5.2",
+    version="1.5.3",
     description="pysaml2 integration for Django",
     long_description=read("README.md"),
     long_description_content_type="text/markdown",