fix: temporarily remove converter, even if it's not the same instance (…

…#642) avoids stackoverflow exceptions Co-authored-by: Konstantin <[email protected]>
Hochfrequenz · Jan 15, 2025 · ea3a4b5 · ea3a4b5
1 parent a183f5f
commit ea3a4b5
Showing 1 changed file with 13 additions and 0 deletions.
diff --git a/BO4E/meta/LenientConverters/LenientSystemTextJsonEnumListConverter.cs b/BO4E/meta/LenientConverters/LenientSystemTextJsonEnumListConverter.cs
@@ -145,6 +145,19 @@ JsonSerializerOptions options
                                     // avoid stack overflow/infinity recursion
                                     optionsWithoutThisConverter.Converters.Remove(this);
                                 }
+
+                                if (
+                                    optionsWithoutThisConverter.Converters.FirstOrDefault(c =>
+                                        c.GetType()
+                                        == typeof(LenientSystemTextJsonEnumListConverter)
+                                    ) is
+                                    { } sameTypeConverter
+                                )
+                                {
+                                    optionsWithoutThisConverter.Converters.Remove(
+                                        sameTypeConverter
+                                    );
+                                }
                                 var helperJsonList = "[" + element.GetRawText() + "]";
                                 var subResultList = System.Text.Json.JsonSerializer.Deserialize(
                                     helperJsonList,