Merge pull request #1422 from DuendeSoftware/joe/6.3-bugfix/error-eve…

…nt-serialization

Joe/6.3 bugfix/error event serialization

src/IdentityServer/Events/UnhandledExceptionEvent.cs

@@ -3,6 +3,7 @@
 
 
 using System;
+using System.Text.Json.Serialization;
 
 namespace Duende.IdentityServer.Events;
 
@@ -41,5 +42,6 @@ public UnhandledExceptionEvent(Exception ex)
     /// <value>
     /// The exception.
     /// </value>
+    [JsonIgnore] // Don't try to serialize exceptions, because System.Text.Json will fail (it doesn't support the pointers inside a call stack)
     public Exception Exception { get; set; }
 }

test/IdentityServer.UnitTests/Events/EventTests.cs

@@ -0,0 +1,45 @@
+// Copyright (c) Duende Software. All rights reserved.
+// See LICENSE in the project root for license information.
+
+
+using System;
+using System.IO;
+using System.Linq;
+using System.Threading.Tasks;
+using Duende.IdentityServer.Configuration;
+using Duende.IdentityServer.Endpoints.Results;
+using Duende.IdentityServer.Models;
+using Duende.IdentityServer.ResponseHandling;
+using Duende.IdentityServer.Validation;
+using FluentAssertions;
+using IdentityModel;
+using UnitTests.Common;
+using Microsoft.AspNetCore.Http;
+using Microsoft.AspNetCore.WebUtilities;
+using Xunit;
+using Duende.IdentityServer.Services;
+using Duende.IdentityServer;
+using Duende.IdentityServer.Events;
+
+namespace UnitTests.Endpoints.Results;
+
+public class EventTests
+{
+
+    [Fact]
+    public void UnhandledExceptionEventCanCallToString()
+    {
+        try
+        {
+            throw new InvalidOperationException("Boom");
+        }
+        catch (Exception ex)
+        {
+            var unhandledExceptionEvent = new UnhandledExceptionEvent(ex);
+
+            var s = unhandledExceptionEvent.ToString();
+
+            s.Should().NotBeNullOrEmpty();
+        }
+    }
+}