JsonNode.ToJsonString(JsonSerializerOptions options) Throw OutOfMemoryException

[saber-wang]

Description

Occasionally throw memory overflow when using JsonNode.ToJsonString

Reproduction Steps

The size of a single JsonObject is within 30000000 bytes

var transObj = new JsonObject();
transObj .ToJsonString(new JsonSerializerOptions(JsonSerializerDefaults.Web)
{
Encoder = System.Text.Encodings.Web.JavaScriptEncoder.UnsafeRelaxedJsonEscaping
}))

Expected behavior

Return string

Actual behavior

Throw OutOfMemoryException

simple repros: https://github.com/saber-wang/JsonObjetcTest

Regression?

No response

Known Workarounds

No response

Configuration

Run in container
base image: mcr.microsoft.com/dotnet/runtime:6.0-alpine

memory configuration

        resources:
          limits:
            memory: 2Gi
          requests:
            memory: 256Mi
        securityContext:
          capabilities:
            add:
            - NET_ADMIN
          privileged: true

Other information

No response

[ghost]

Tagging subscribers to this area: @dotnet/area-system-text-json, @gregsdennis
See info in area-owners.md if you want to be subscribed.

Issue Details

---

Description

Occasionally throw memory overflow when using JsonNode.ToJsonString

Reproduction Steps

The size of a single JsonObject is within 30000000 bytes

var transObj = new JsonObject();
transObj .ToJsonString(new JsonSerializerOptions(JsonSerializerDefaults.Web)
{
Encoder = System.Text.Encodings.Web.JavaScriptEncoder.UnsafeRelaxedJsonEscaping
}))

Expected behavior

Return string

Actual behavior

Throw OutOfMemoryException

dumps: https://github.com/saber-wang/dumps

Regression?

No response

Known Workarounds

No response

Configuration

Run in container
base image: mcr.microsoft.com/dotnet/runtime:6.0-alpine

Other information

No response

Author:	saber-wang
Assignees:	-
Labels:	area-System.Text.Json, untriaged
Milestone:	-

[saber-wang]

@mangod9

[mangod9]

hi @saber-wang, could you also please provide what memory configuration you are using for the container? If you possibly have a sample json input that might help too. thanks

[danmoseley]

Please if you can share minimal, runnable code that repros this.

[saber-wang]

@mangod9

         limits:
            memory: 2Gi

Sample JSON I try to print it out the next time the memory overflows

[mangod9]

Hi @saber-wang, did you mean to close this issue? If so was it resolved on your end?

[saber-wang]

@mangod9 I got the JSON and tested it and found that the memory overflow is in line with expectations. However, the memory used by jsonnode is more than twice that of the original JSON. In the future, I may not use jsonnode but directly process strings

I can provide a simple repros if necessary

[saber-wang]

@mangod9 A simple example https://github.com/saber-wang/JsonObjetcTest