Add logging to get a better understanding of production errors

oidc/viewsets.py

@@ -4,6 +4,8 @@
 
 import importlib
 import re
+import traceback
+import logging
 from typing import Optional, Tuple
 
 from django.conf import settings
@@ -37,6 +39,8 @@
 default_config = getattr(default, "OPENID_CONNECT_VIEWSET_CONFIG", {})
 SSO_COOKIE_NAME = "SSO"
 
+logger = logging.getLogger(__name__)
+
 
 class BaseOpenIDConnectViewset(viewsets.ViewSet):
     """
@@ -201,6 +205,7 @@ def validate_fields(self, data: dict) -> dict:
                 field_validation_regex = self.field_validation_regex[k]
                 regex = re.compile(field_validation_regex.get("regex"))
                 if regex and not regex.search(data[k]):
+                    logger.info(f"Invalid `{k}` value `{data[k]}`")
                     raise ValueError(
                         field_validation_regex.get("help_text")
                         or f"Invalid `{k}` value `{data[k]}`"
@@ -321,11 +326,13 @@ def callback(self, request: HttpRequest, **kwargs: dict) -> HttpResponse:  # noq
                                 and list(missing_fields)[0] == "username"
                             ):
                                 data = {"id_token": id_token}
+                                logger.info("missing_fields: ", missing_fields)
                                 return Response(
                                     data, template_name="oidc/oidc_user_data_entry.html"
                                 )
                             else:
                                 missing_fields = ", ".join(missing_fields)
+                                logger.error("missing fields", missing_fields)
                                 return Response(
                                     {
                                         "error": _(
@@ -343,6 +350,7 @@ def callback(self, request: HttpRequest, **kwargs: dict) -> HttpResponse:  # noq
                                     "id_token": id_token,
                                     "error": f"{field.capitalize()} field is already in use.",
                                 }
+                                logger.info(data)
                                 return Response(
                                     data, template_name="oidc/oidc_user_data_entry.html"
                                 )
@@ -356,6 +364,9 @@ def callback(self, request: HttpRequest, **kwargs: dict) -> HttpResponse:  # noq
 
                         user = self.create_login_user(create_data)
                 except ValueError as e:
+                    stack_trace = traceback.format_exc()
+                    logger.info("ValueError")
+                    logger.info(stack_trace)
                     return Response(
                         {"error": str(e), "id_token": id_token},
                         status=status.HTTP_400_BAD_REQUEST,