fix: propogate error_message to avoid silently dropping it

Signed-off-by: Danju Visvanathan <danju.visvanathan@gmail.com>

Author: danjuv

openfeature/hook/logging_hook.py

@@ -59,10 +59,10 @@ def error(
         self, hook_context: HookContext, exception: Exception, hints: HookHints
     ) -> None:
         args = self._build_args(hook_context, "error")
-        args["error_code"] = (
-            exception.error_code
-            if isinstance(exception, OpenFeatureError)
-            else ErrorCode.GENERAL
-        )
-        args["error_message"] = str(exception)
+        if isinstance(exception, OpenFeatureError):
+            args["error_code"] = exception.error_code
+            args["error_message"] = exception.error_message
+        else:
+            args["error_code"] = ErrorCode.GENERAL
+            args["error_message"] = str(exception)
         self.logger.error("Flag evaluation %s", args)

tests/hook/test_logging_hook.py

@@ -126,7 +126,7 @@ def test_error_extracts_error_code_from_open_feature_error(hook_context):
             "domain": "my-domain",
             "provider_name": "my-provider",
             "error_code": ErrorCode.FLAG_NOT_FOUND,
-            "error_message": str(exception),
+            "error_message": "flag not found",
         },
     )